diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-30 19:03:10 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-30 19:03:10 +0000 |
| commit | 8cf4c691e16a2696ab967287b6c0020c76042ab8 (patch) | |
| tree | 836e03fda0ec4a44b782cd329dfa808808be4c0f /chrome/browser/gtk | |
| parent | a88fdf531a6a3f34e22d3d8443516b3970e4362f (diff) | |
| download | chromium_src-8cf4c691e16a2696ab967287b6c0020c76042ab8.zip chromium_src-8cf4c691e16a2696ab967287b6c0020c76042ab8.tar.gz chromium_src-8cf4c691e16a2696ab967287b6c0020c76042ab8.tar.bz2 | |
Add some space to the browser actions toolbar.
This adds space between the button(s) and the overflow chevron (or the separator if there is no overflow menu). It makes the spacing match windows on the right (the left is kind of hard to judge since windows now hides the gripper).
BUG=47487
TEST=manual
Review URL: http://codereview.chromium.org/2857031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51285 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk')
| -rw-r--r-- | chrome/browser/gtk/browser_actions_toolbar_gtk.cc | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc index f19c720..b1126bc 100644 --- a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc +++ b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc @@ -44,6 +44,10 @@ const int kButtonSize = 29; // use their maximum allowed size. const int kButtonPadding = 3; +// The padding to the right of the browser action buttons (between the buttons +// and the separator, or chevron if it's showing). +const int kPaddingToRightOfButtons = 5; + // The padding to the left, top and bottom of the browser actions toolbar // separator. const int kSeparatorPadding = 2; @@ -370,8 +374,13 @@ BrowserActionsToolbarGtk::BrowserActionsToolbarGtk(Browser* browser) signals_.Connect(overflow_button_.widget(), "button-press-event", G_CALLBACK(OnOverflowButtonPressThunk), this); + // Add some blank space on the right of the browser action buttons. + GtkWidget* spacer = gtk_alignment_new(0, 0, 1, 1); + gtk_widget_set_size_request(spacer, kPaddingToRightOfButtons, -1); + gtk_box_pack_start(GTK_BOX(hbox_.get()), gripper, FALSE, FALSE, 0); gtk_box_pack_start(GTK_BOX(hbox_.get()), button_hbox_.get(), TRUE, TRUE, 0); + gtk_box_pack_start(GTK_BOX(hbox_.get()), spacer, FALSE, FALSE, 0); gtk_box_pack_start(GTK_BOX(hbox_.get()), overflow_button_.widget(), FALSE, FALSE, 0); gtk_box_pack_start(GTK_BOX(hbox_.get()), separator_, FALSE, FALSE, 0); |