I was having trouble making an attachment to bugzilla.
I made a patch for this issue, but the ordering is kinda outa wack in the menu:

Here is a patch that fixes the issue, but because of the way the overlay window is created, it can't easily be made to match the order of the menubar. Modifying the order of the menubar results in the numbers not being in order! Do people mind if we change the keyboard shortcuts so the numbering is in order?


Pierre


[EMAIL PROTECTED] wrote:
http://bugs.cinelerra.org/show_bug.cgi?id=360

           Summary: consistent list of items in View & Show Overlays
           Product: Cinelerra
           Version: 2.1
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: enhancement
          Priority: Medium
         Component: User Interface
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


Hi!

The Program --> View show the list of different menu items (Show assets, Show
titles) which is not in the same order as the items in Program --> Window -->
Show Overlays, so the feature request is to put the two in the same order to
improve consistency in the user-interface.

Sincerely,
Gour



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

_______________________________________________
Cinelerra mailing list
[email protected]
https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra

Index: cinelerra/mainmenu.C
===================================================================
--- cinelerra/mainmenu.C	(revision 957)
+++ cinelerra/mainmenu.C	(working copy)
@@ -171,11 +171,11 @@
 	viewmenu->add_item(show_assets = new ShowAssets(mwindow, "0"));
 	viewmenu->add_item(show_titles = new ShowTitles(mwindow, "1"));
 	viewmenu->add_item(show_transitions = new ShowTransitions(mwindow, "2"));
+	viewmenu->add_item(plugin_automation = new PluginAutomation(mwindow, "7"));
 	viewmenu->add_item(fade_automation = new ShowAutomation(mwindow, _("Fade"), "3", AUTOMATION_FADE));
 	viewmenu->add_item(mute_automation = new ShowAutomation(mwindow, _("Mute"), "4", AUTOMATION_MUTE));
 	viewmenu->add_item(mode_automation = new ShowAutomation(mwindow, _("Mode"), "5", AUTOMATION_MODE));
 	viewmenu->add_item(pan_automation = new ShowAutomation(mwindow, _("Pan"), "6", AUTOMATION_PAN));
-	viewmenu->add_item(plugin_automation = new PluginAutomation(mwindow, "7"));
 	viewmenu->add_item(mask_automation = new ShowAutomation(mwindow, _("Mask"), "8", AUTOMATION_MASK));
 	viewmenu->add_item(camera_x = new ShowAutomation(mwindow, _("Camera X"), "", AUTOMATION_CAMERA_X));
 	viewmenu->add_item(camera_y = new ShowAutomation(mwindow, _("Camera Y"), "", AUTOMATION_CAMERA_Y));
Index: cinelerra/viewmenu.C
===================================================================
--- cinelerra/viewmenu.C	(revision 952)
+++ cinelerra/viewmenu.C	(working copy)
@@ -122,7 +122,7 @@
 
 
 PluginAutomation::PluginAutomation(MWindow *mwindow, char *hotkey)
- : BC_MenuItem(_("Plugin keyframes"), hotkey, hotkey[0]) 
+ : BC_MenuItem(_("Plugin Autos"), hotkey, hotkey[0]) 
 { 
 	this->mwindow = mwindow; 
 }
Index: cinelerra/gwindowgui.C
===================================================================
--- cinelerra/gwindowgui.C	(revision 952)
+++ cinelerra/gwindowgui.C	(working copy)
@@ -55,6 +55,22 @@
 	"Nudge"
 };
 
+static int auto_reorder[] = 
+{
+	AUTOMATION_FADE,
+	AUTOMATION_MUTE,
+	AUTOMATION_MODE,
+	AUTOMATION_PAN,
+	AUTOMATION_MASK,
+	AUTOMATION_CAMERA_X,
+	AUTOMATION_CAMERA_Y,
+	AUTOMATION_CAMERA_Z,
+	AUTOMATION_PROJECTOR_X,
+	AUTOMATION_PROJECTOR_Y,
+	AUTOMATION_PROJECTOR_Z
+};
+
+
 void GWindowGUI::calculate_extents(BC_WindowBase *gui, int *w, int *h)
 {
 	int temp1, temp2, temp3, temp4, temp5, temp6, temp7;
@@ -127,9 +143,9 @@
 			this, 
 			x, 
 			y, 
-			i,
+			auto_reorder[i],
 			-1, 
-			auto_text[i]));
+			auto_text[auto_reorder[i]]));
 		y += auto_toggle[i]->get_h() + 5;
 	}
 }

Reply via email to