summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--gtk2_ardour/route_time_axis.cc13
-rw-r--r--gtk2_ardour/route_time_axis.h2
-rw-r--r--gtk2_ardour/route_ui.cc6
-rw-r--r--gtk2_ardour/route_ui.h2
4 files changed, 10 insertions, 13 deletions
diff --git a/gtk2_ardour/route_time_axis.cc b/gtk2_ardour/route_time_axis.cc
index 73dcf5d9a3..957dfcd88f 100644
--- a/gtk2_ardour/route_time_axis.cc
+++ b/gtk2_ardour/route_time_axis.cc
@@ -441,7 +441,7 @@ RouteTimeAxisView::build_display_menu ()
MenuList& items = display_menu->items();
display_menu->set_name ("ArdourContextMenu");
- items.push_back (MenuElem (_("Color..."), sigc::mem_fun(*this, &RouteTimeAxisView::select_track_color)));
+ items.push_back (MenuElem (_("Color..."), sigc::hide_return (sigc::mem_fun (*this, &RouteUI::choose_color))));
build_size_menu ();
items.push_back (MenuElem (_("Height"), *_size_menu));
@@ -863,13 +863,12 @@ RouteTimeAxisView::set_height (uint32_t h)
}
void
-RouteTimeAxisView::select_track_color ()
+RouteTimeAxisView::set_color (Gdk::Color const & c)
{
- if (RouteUI::choose_color ()) {
-
- if (_view) {
- _view->apply_color (_color, StreamView::RegionColor);
- }
+ RouteUI::set_color (c);
+
+ if (_view) {
+ _view->apply_color (_color, StreamView::RegionColor);
}
}
diff --git a/gtk2_ardour/route_time_axis.h b/gtk2_ardour/route_time_axis.h
index 8515f375b9..54d06b4135 100644
--- a/gtk2_ardour/route_time_axis.h
+++ b/gtk2_ardour/route_time_axis.h
@@ -212,7 +212,7 @@ protected:
void reset_samples_per_unit ();
void horizontal_position_changed ();
- void select_track_color();
+ void set_color (Gdk::Color const &);
virtual void build_automation_action_menu ();
virtual void append_extra_display_menu_items () {}
diff --git a/gtk2_ardour/route_ui.cc b/gtk2_ardour/route_ui.cc
index b0338ff121..209210b8ac 100644
--- a/gtk2_ardour/route_ui.cc
+++ b/gtk2_ardour/route_ui.cc
@@ -1258,12 +1258,10 @@ RouteUI::toggle_solo_safe (Gtk::CheckMenuItem* check)
}
bool
-RouteUI::choose_color()
+RouteUI::choose_color ()
{
bool picked;
- Gdk::Color color;
-
- color = Gtkmm2ext::UI::instance()->get_color (_("Color Selection"), picked, &_color);
+ Gdk::Color const color = Gtkmm2ext::UI::instance()->get_color (_("Color Selection"), picked, &_color);
if (picked) {
set_color (color);
diff --git a/gtk2_ardour/route_ui.h b/gtk2_ardour/route_ui.h
index ff133642a2..c59ace3061 100644
--- a/gtk2_ardour/route_ui.h
+++ b/gtk2_ardour/route_ui.h
@@ -77,7 +77,7 @@ class RouteUI : public virtual AxisView
boost::shared_ptr<ARDOUR::Route> _route;
- void set_color (const Gdk::Color & c);
+ virtual void set_color (const Gdk::Color & c);
bool choose_color ();
bool ignore_toggle;