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

Pixelpushing: align toolbar buttons

Browse Source 
Removing the shadow makes toolbar and editor-tool buttons align
when the editor-mixer is hidden. It also improves consistency
with all other tabs that have no shadow on the left.

Increasing the table border width to 1, makes aligns the
editor-mixer button with the main toolbar.
This commit is contained in:
Robin Gareus 2022-10-18 01:37:41 +02:00
parent 7c6c6290ee
commit 452eedc994
Signed by: rgareus
GPG Key ID: A090BCE02CF57F04
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gtk2_ardour/ardour_ui2.cc
View File

@ -506,7 +506,7 @@ ARDOUR_UI::setup_transport ()
/* top level packing */ /* top level packing */
transport_table.set_spacings (0); transport_table.set_spacings (0);
transport_table.set_row_spacings (4); transport_table.set_row_spacings (4);
transport_table.set_border_width (0); transport_table.set_border_width (1);
transport_frame.set_name ("TransportFrame"); transport_frame.set_name ("TransportFrame");
transport_frame.set_shadow_type (Gtk::SHADOW_NONE); transport_frame.set_shadow_type (Gtk::SHADOW_NONE);

6
gtk2_ardour/editor.cc
View File

@ -653,12 +653,14 @@ Editor::Editor ()
time_bars_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK); time_bars_event_box.set_events (Gdk::BUTTON_PRESS_MASK|Gdk::BUTTON_RELEASE_MASK);
time_bars_event_box.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_label_button_release)); time_bars_event_box.signal_button_release_event().connect (sigc::mem_fun(*this, &Editor::ruler_label_button_release));
#ifndef MIXBUS
ArdourWidgets::ArdourDropShadow *axis_view_shadow = manage (new (ArdourWidgets::ArdourDropShadow)); ArdourWidgets::ArdourDropShadow *axis_view_shadow = manage (new (ArdourWidgets::ArdourDropShadow));
axis_view_shadow->set_size_request (4, -1); axis_view_shadow->set_size_request (4, -1);
axis_view_shadow->set_name("EditorWindow"); axis_view_shadow->set_name("EditorWindow");
axis_view_shadow->show(); axis_view_shadow->show();
edit_packer.attach (*axis_view_shadow, 0, 1, 0, 2, FILL, FILL|EXPAND, 0, 0); edit_packer.attach (*axis_view_shadow, 0, 1, 0, 2, FILL, FILL|EXPAND, 0, 0);
#endif
/* labels for the time bars */ /* labels for the time bars */
edit_packer.attach (time_bars_event_box, 1, 2, 0, 1, FILL, SHRINK, 0, 0); edit_packer.attach (time_bars_event_box, 1, 2, 0, 1, FILL, SHRINK, 0, 0);
@ -3342,13 +3344,15 @@ Editor::setup_toolbar ()
/* Pack everything in... */ /* Pack everything in... */
toolbar_hbox.set_spacing (2); toolbar_hbox.set_spacing (2);
toolbar_hbox.set_border_width (2); toolbar_hbox.set_border_width (1);
#ifndef MIXBUS
ArdourWidgets::ArdourDropShadow *tool_shadow = manage (new (ArdourWidgets::ArdourDropShadow)); ArdourWidgets::ArdourDropShadow *tool_shadow = manage (new (ArdourWidgets::ArdourDropShadow));
tool_shadow->set_size_request (4, -1); tool_shadow->set_size_request (4, -1);
tool_shadow->show(); tool_shadow->show();
ebox_hpacker.pack_start (*tool_shadow, false, false); ebox_hpacker.pack_start (*tool_shadow, false, false);
#endif
ebox_hpacker.pack_start(ebox_vpacker, true, true); ebox_hpacker.pack_start(ebox_vpacker, true, true);
Gtk::EventBox* spacer = manage (new Gtk::EventBox); // extra space under the mouse toolbar, for aesthetics Gtk::EventBox* spacer = manage (new Gtk::EventBox); // extra space under the mouse toolbar, for aesthetics