allanger/ardour
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Activity

rationalize region "show properties" and "edit in separate window" menu items

Browse Source
This commit is contained in:
Paul Davis
2025-12-01 20:48:35 -07:00
parent fd0e9e840e
commit 7a5591121b
5 changed files with 8 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gtk2_ardour/ardour.menus.in
View File

@@ -385,10 +385,11 @@
<separator/>
<menuitem action='loop-region'/>
<menuitem action='rename-region'/>
<menuitem action='show-region-properties'/>
<menuitem action='loudness-analyze-region'/>
<menuitem action='spectral-analyze-region'/>
<menu action='RegionMenuEdit'>
<menuitem action='edit-region-dedicated-window'/>
<separator/>
#ifndef MIXBUS
<menuitem action='combine-regions'/>
<menuitem action='uncombine-regions'/>
@@ -925,8 +926,7 @@
<separator/>
<menuitem action='loop-region'/>
<menuitem action='rename-region'/>
<menuitem action='show-region-properties'/>
<menuitem action='edit-region-pianoroll-window'/>
<menuitem action='edit-region-dedicated-window'/>
<menu action='RegionMenuEdit'>
<menuitem action='combine-regions'/>
<menuitem action='uncombine-regions'/>

3
gtk2_ardour/editor.h
View File

@@ -229,7 +229,7 @@ public:
bool extend_selection_to_track (TimeAxisView&);
void edit_region_in_pianoroll_window ();
void edit_region_in_dedicated_window ();
void maybe_edit_region_in_bottom_pane (RegionView&);
void play_selection ();
@@ -1156,7 +1156,6 @@ private:
void remove_selected_regions ();
void remove_regions (const RegionSelection&, bool can_ripple, bool as_part_of_other_command);
void remove_clicked_region ();
void show_region_properties ();
void show_midi_list_editor ();
void rename_region ();
void duplicate_some_regions (RegionSelection&, float times);

5
gtk2_ardour/editor_actions.cc
View File

@@ -1417,11 +1417,8 @@ Editor::register_region_actions ()
/* Open the list editor dialogue for the selected regions */
register_region_action (_region_actions, RegionActionTarget (SelectedRegions), "show-region-list-editor", _("List Editor..."), sigc::mem_fun (*this, &Editor::show_midi_list_editor));
/* Open the region properties dialogue for the selected regions */
register_region_action (_region_actions, RegionActionTarget (SelectedRegions), "show-region-properties", _("Properties..."), sigc::mem_fun (*this, &Editor::show_region_properties));
/* Edit the region in a separate region pianoroll window */
register_region_action (_region_actions, RegionActionTarget (SelectedRegions), "edit-region-pianoroll-window", _("Edit in separate window..."), sigc::mem_fun (*this, &Editor::edit_region_in_pianoroll_window));
register_region_action (_region_actions, RegionActionTarget (SelectedRegions), "edit-region-dedicated-window", _("Edit in separate window..."), sigc::mem_fun (*this, &Editor::edit_region_in_dedicated_window));
register_region_action (_region_actions, RegionActionTarget (SelectedRegions|EnteredRegions), "play-selected-regions", _("Play Selected Regions"), sigc::mem_fun(*this, &Editor::play_selected_region));
register_region_action (_region_actions, RegionActionTarget (SelectedRegions|EnteredRegions), "tag-selected-regions", _("Tag Selected Regions"), sigc::mem_fun(*this, &Editor::tag_selected_region));

2
gtk2_ardour/editor_mouse.cc
View File

@@ -1346,7 +1346,7 @@ Editor::button_release_handler (ArdourCanvas::Item* item, GdkEvent* event, ItemT
if (!_drags->active () && Keyboard::is_edit_event (&event->button)) {
switch (item_type) {
case RegionItem:
show_region_properties ();
edit_region_in_dedicated_window ();
break;
case TempoMarkerItem: {
ArdourMarker* marker;

11
gtk2_ardour/editor_ops.cc
View File

@@ -2757,13 +2757,6 @@ Editor::lower_region_to_bottom ()
do_layer_operation (LowerToBottom);
}
/** Show the region editor for the selected regions */
void
Editor::show_region_properties ()
{
selection->foreach_regionview (&RegionView::show_region_editor);
}
/** Show the midi list editor for the selected MIDI regions */
void
Editor::show_midi_list_editor ()
@@ -9520,9 +9513,9 @@ Editor::temporal_zoom_extents ()
}
void
Editor::edit_region_in_pianoroll_window ()
Editor::edit_region_in_dedicated_window ()
{
selection->foreach_midi_regionview (&MidiRegionView::show_region_editor);
selection->foreach_regionview (&RegionView::show_region_editor);
}
void