From 8e4e7fde05c34dd09b73494df5b8d8e588dae02c Mon Sep 17 00:00:00 2001 From: Ben Loftis Date: Mon, 12 Feb 2018 11:42:21 -0600 Subject: [PATCH] Re-pack Editor Toolbar: Move zoom tools to right of toolbar. Tweak numeric (mode) keybindings to more closely match the order they appear in the UI. Add some new default keybindings. --- gtk2_ardour/ardour.keys.in | 28 ++++++++++++++++------------ gtk2_ardour/ardour.menus.in | 1 + gtk2_ardour/editor.cc | 31 ++++++++++++------------------- 3 files changed, 29 insertions(+), 31 deletions(-) diff --git a/gtk2_ardour/ardour.keys.in b/gtk2_ardour/ardour.keys.in index 78b34f6e70..96b1785776 100644 --- a/gtk2_ardour/ardour.keys.in +++ b/gtk2_ardour/ardour.keys.in @@ -102,8 +102,6 @@ This mode provides many different operations on both regions and control points, @vis|Editor/zoom-to-session|<@TERTIARY@>underscore|zoom to session @eep|Editor/edit-cursor-to-previous-region-sync|apostrophe|EP to prev region sync @eep|Editor/edit-cursor-to-next-region-sync|semicolon|EP to next region sync -@eep|Editor/cycle-edit-point|grave|next EP w/o marker -@eep|Editor/cycle-edit-point-with-marker|<@PRIMARY@>grave|next EP w/marker @trans|Transport/ToggleRoll|space|toggle roll @trans|Transport/PlaySelection|<@SECONDARY@>space|play edit range @@ -174,7 +172,6 @@ This mode provides many different operations on both regions and control points, @wvis|Window/toggle-session-options-editor|<@SECONDARY@>o|toggle preferences dialog @edit|Editor/set-playhead|p|set playhead position @wvis|Window/toggle-audio-connection-manager|<@SECONDARY@>p|toggle global audio patchbay -@midi|MIDI/panic|<@PRIMARY@><@SECONDARY@>p|MIDI panic (stop all notes etc) ;; MIDDLE ROW @@ -194,8 +191,8 @@ This mode provides many different operations on both regions and control points, @select|Editor/select-all-in-punch-range|<@PRIMARY@>d|select all in punch range @vis|Editor/fit-selection|f|fit selection vertically @edit|Editor/toggle-follow-playhead|<@PRIMARY@>f|toggle playhead tracking +@edit|Editor/toggle-stationary-playhead|<@TERTIARY@>f|toggle stationary playhead @rop|Region/show-rhythm-ferret|<@SECONDARY@>f|show rhythm ferret window -@gmode|Transport/ToggleFollowEdits|<@TERTIARY@>f|toggle playhead follows edits @wvis|Common/ToggleMaximalEditor|<@PRIMARY@><@SECONDARY@>f|maximise editor space @wvis|Common/ToggleMaximalMixer|<@PRIMARY@><@TERTIARY@>f|maximise mixer space @edit|Region/play-selected-regions|h|play selected region(s) @@ -340,14 +337,21 @@ This mode provides many different operations on both regions and control points, ;; numbers -@vis|Editor/cycle-zoom-focus|1|cycle to next zoom focus -@grid|Editor/cycle-snap-mode|2|cycle to next snap mode -@grid|Editor/prev-grid-choice|3|use prev grid unit -@grid|Editor/next-grid-choice|4|use next grid unit -@gmode|Transport/ToggleAutoPlay|5|toggle auto play -@gmode|Transport/ToggleAutoReturn|6|toggle auto return -@gmode|Transport/ToggleClick|7|toggle click (metronome) -@mmode|MouseMode/set-mouse-mode-object-range|8|link object/range tools +@gmode|Transport/ToggleClick|grave|toggle click (metronome) +@midi|MIDI/panic|<@PRIMARY@>grave|MIDI panic (stop all midi notes) +@trans|Transport/ToggleExternalSync|<@SECONDARY@>grave|toggle external sync +@eep|Editor/cycle-edit-mode|1|cycle edit mode {slide, lock, ripple} +@eep|Editor/cycle-edit-point|2|next EP w/o marker {playhead, mouse} +@eep|Editor/cycle-edit-point-with-marker|<@PRIMARY@>2|next EP w/marker {playhead, mouse, marker} +@mmode|MouseMode/set-mouse-mode-object-range|3|Smart Mode (provide some Range tools in Object mode) +@gmode|Transport/ToggleFollowEdits|<@PRIMARY@>3|toggle playhead follows edits +@grid|Editor/cycle-snap-mode|4|cycle to next snap mode {On/Off} +@grid|Editor/prev-grid-choice|5|use prev grid unit +@grid|Editor/next-grid-choice|6|use next grid unit +@gmode|Transport/ToggleAutoReturn|7|toggle auto return +@gmode|Transport/ToggleAutoPlay|<@PRIMARY@>7|toggle auto play +@gmode|Transport/TogglePunch|8|toggle punch in+out +; ToDo @gmode|Transport/ToggleRecMode|<@PRIMARY@>8|toggle recording mode ( layered, overwrite, merge ) @ranges|Region/set-tempo-from-region|9|set tempo (1 bar) from region(s) @ranges|Editor/set-tempo-from-edit-range|0|set tempo (1 bar) from edit range diff --git a/gtk2_ardour/ardour.menus.in b/gtk2_ardour/ardour.menus.in index 210c29dbbe..22e06be038 100644 --- a/gtk2_ardour/ardour.menus.in +++ b/gtk2_ardour/ardour.menus.in @@ -158,6 +158,7 @@ + diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc index b79557f849..748eb6d23b 100644 --- a/gtk2_ardour/editor.cc +++ b/gtk2_ardour/editor.cc @@ -2950,6 +2950,9 @@ Editor::setup_toolbar () if (!ARDOUR::Profile->get_trx()) { mode_box->pack_start (edit_mode_selector, false, false); + mode_box->pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); + mode_box->pack_start (edit_point_selector, false, false); + mode_box->pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); } mode_box->pack_start (*mouse_mode_box, false, false); @@ -3037,13 +3040,6 @@ Editor::setup_toolbar () snap_box.pack_start (snap_mode_button, false, false); snap_box.pack_start (grid_type_selector, false, false); - /* Edit Point*/ - HBox *ep_box = manage (new HBox); - ep_box->set_spacing (2); - ep_box->set_border_width (2); - - ep_box->pack_start (edit_point_selector, false, false); - /* Nudge */ HBox *nudge_box = manage (new HBox); @@ -3069,23 +3065,20 @@ Editor::setup_toolbar () toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); - toolbar_hbox.pack_start (_zoom_box, false, false); - - toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); - - toolbar_hbox.pack_start (_track_box, false, false); - - toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); - toolbar_hbox.pack_start (snap_box, false, false); toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); - toolbar_hbox.pack_start (*ep_box, false, false); - - toolbar_hbox.pack_start (*(manage (new ArdourVSpacer ())), false, false, 3); - toolbar_hbox.pack_start (*nudge_box, false, false); + + //zoom tools on right ege + + toolbar_hbox.pack_end (_zoom_box, false, false); + + toolbar_hbox.pack_end (*(manage (new ArdourVSpacer ())), false, false, 3); + + toolbar_hbox.pack_end (_track_box, false, false); + } toolbar_hbox.show_all ();