diff options
author | Carl Hetherington <carl@carlh.net> | 2012-01-24 00:53:18 +0000 |
---|---|---|
committer | Carl Hetherington <carl@carlh.net> | 2012-01-24 00:53:18 +0000 |
commit | 6918bba1708242ce0c06a37a6e97ab10a3e8d044 (patch) | |
tree | 22c9aa771308d9458d9267001be39fd454643aee /gtk2_ardour/editor.cc | |
parent | 5248d79fa4d153c209b416e69b5e864b2c45c7a1 (diff) |
Use the same criteron to sensitize the choose-top-region menu item as to decide whether to open it or not, to prevent a sensitive menu item that does nothing.
git-svn-id: svn://localhost/ardour2/branches/3.0@11322 d708f5d6-7413-0410-9779-e7cbd77b26cf
Diffstat (limited to 'gtk2_ardour/editor.cc')
-rw-r--r-- | gtk2_ardour/editor.cc | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc index f679eecca6..9bf99a40e2 100644 --- a/gtk2_ardour/editor.cc +++ b/gtk2_ardour/editor.cc @@ -1807,17 +1807,10 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items, boost::sha _popup_region_menu_item->set_label (menu_item_name); } - /* Use the mouse position rather than the edit point to decide whether to show the `choose top region' - dialogue. If we use the edit point it gets a bit messy because the user still has to click over - *some* region in order to get the region context menu stuff to be displayed at all. - */ - - framepos_t mouse; - bool ignored; - mouse_frame (mouse, ignored); + const framepos_t position = get_preferred_edit_position (false, true); edit_items.push_back (*_popup_region_menu_item); - if (track->playlist()->count_regions_at (mouse) > 1 && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) { + if (track->playlist()->count_regions_at (position) > 1 && (layering_order_editor == 0 || !layering_order_editor->is_visible ())) { edit_items.push_back (*manage (_region_actions->get_action ("choose-top-region-context-menu")->create_menu_item ())); } edit_items.push_back (SeparatorElem()); |