ardour/ardour
ardour/ardour
13
1
Fork 0
Code Pull Requests Releases Activity

Replace positioning function with popup helper

Browse Source 
Because all uses of the function positioning menus anchored to a widget
were as callback argument to Gtk::Menu::popup() where the caller needed
to correctly bind arguments, this led to repeated and a bit obscure code.

Wrap the logic into an helper function that takes care of all that, and
update the callers.
This commit is contained in:
Julien "_FrnchFrgg_" RIVAUD 2016-08-16 01:57:58 +02:00
parent 46710a75de
commit 6a985df81e
5 changed files with 21 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
gtk2_ardour/ardour_dropdown.cc
View File

@ -67,17 +67,11 @@ ArdourDropdown::menu_size_request(Requisition *req) {
req->width = max(req->width, get_allocation().get_width());
}
void
ArdourDropdown::position_menu(int& x, int& y, bool& push_in) {
Gtkmm2ext::position_menu_anchored (&_menu, this, get_text(), x, y, push_in);
}
bool
ArdourDropdown::on_button_press_event (GdkEventButton* ev)
{
if (ev->type == GDK_BUTTON_PRESS) {
_menu.popup (sigc::mem_fun(this, &ArdourDropdown::position_menu),
1, ev->time);
Gtkmm2ext::anchored_menu_popup(&_menu, this, get_text(), 1, ev->time);
}
return true;
}

1
gtk2_ardour/ardour_dropdown.h
View File

@ -38,7 +38,6 @@ class ArdourDropdown : public ArdourButton
bool on_button_press_event (GdkEventButton*);
bool on_scroll_event (GdkEventScroll*);
void position_menu(int&, int&, bool&);
void menu_size_request(Gtk::Requisition*);
void clear_items ();

4
gtk2_ardour/generic_pluginui.cc
View File

@ -955,9 +955,7 @@ GenericPluginUI::astate_clicked (ControlUI* cui)
items.push_back (MenuElem (_("Touch"),
sigc::bind (sigc::mem_fun(*this, &GenericPluginUI::set_automation_state), (AutoState) Touch, cui)));
automation_menu->popup (
boost::bind (&Gtkmm2ext::position_menu_anchored, automation_menu, &cui->automate_button, "", _1, _2, _3),
1, gtk_get_current_event_time());
anchored_menu_popup(automation_menu, &cui->automate_button, "", 1, gtk_get_current_event_time());
}
void

8
libs/gtkmm2ext/gtkmm2ext/utils.h
View File

@ -97,10 +97,10 @@ namespace Gtkmm2ext {
int clip_height,
Gdk::Color fg);
LIBGTKMM2EXT_API void position_menu_anchored (const Gtk::Menu* const menu,
Gtk::Widget* const anchor,
const std::string& selected,
int& x, int& y, bool& push_in);
LIBGTKMM2EXT_API void anchored_menu_popup (Gtk::Menu* const menu,
Gtk::Widget* const anchor,
const std::string& selected,
guint button, guint32 time);
LIBGTKMM2EXT_API void set_popdown_strings (Gtk::ComboBoxText&,
const std::vector<std::string>&);

18
libs/gtkmm2ext/utils.cc
View File

@ -310,10 +310,10 @@ Gtkmm2ext::pixbuf_from_string(const string& name, const Pango::FontDescription&
}
void
Gtkmm2ext::position_menu_anchored (const Gtk::Menu* const menu,
_position_menu_anchored (int& x, int& y, bool& push_in,
const Gtk::Menu* const menu,
Gtk::Widget* const anchor,
const std::string& selected,
int& x, int& y, bool& push_in) {
const std::string& selected) {
using namespace Gdk;
using namespace Gtk;
using namespace Gtk::Menu_Helpers;
@ -416,6 +416,18 @@ Gtkmm2ext::position_menu_anchored (const Gtk::Menu* const menu,
push_in = false;
}
void
Gtkmm2ext::anchored_menu_popup (Gtk::Menu* const menu,
Gtk::Widget* const anchor,
const std::string& selected,
guint button, guint32 time) {
menu->popup(
sigc::bind (sigc::ptr_fun(&_position_menu_anchored),
menu, anchor, selected),
button,
time);
}
void
Gtkmm2ext::set_popdown_strings (Gtk::ComboBoxText& cr, const vector<string>& strings)
{