diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-22 01:35:13 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-22 01:35:13 +0000 |
| commit | c8944b2e4f318c82fc71da5cc29b50bfb86e6868 (patch) | |
| tree | e330b6b1b2f10127a204426ea89870d8f878bb27 /chrome/browser | |
| parent | 5870da810ab64cbc2510172aef3727333397a671 (diff) | |
| download | chromium_src-c8944b2e4f318c82fc71da5cc29b50bfb86e6868.zip chromium_src-c8944b2e4f318c82fc71da5cc29b50bfb86e6868.tar.gz chromium_src-c8944b2e4f318c82fc71da5cc29b50bfb86e6868.tar.bz2 | |
GTK: Center the tab text by its meanline, rather than halfway between its ascent and descent.
The tab text was previously one pixel too high. With this patch it matches Windows.
I think before Evan's recent font size patch, this issue was hidden by favorable rounding.
Review URL: http://codereview.chromium.org/159169
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21256 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/gtk/tabs/tab_renderer_gtk.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/gtk/tabs/tab_renderer_gtk.cc b/chrome/browser/gtk/tabs/tab_renderer_gtk.cc index d5e2dda..a2d0d7f 100644 --- a/chrome/browser/gtk/tabs/tab_renderer_gtk.cc +++ b/chrome/browser/gtk/tabs/tab_renderer_gtk.cc @@ -502,7 +502,9 @@ void TabRendererGtk::Layout() { if (!is_pinned() || width() >= kPinnedTabRendererAsTabWidth) { // Size the Title text to fill the remaining space. int title_left = favicon_bounds_.right() + kFavIconTitleSpacing; - int title_top = kTopPadding + (content_height - title_font_height_) / 2; + // Center the mean line of the text in the content area. + int title_top = kTopPadding + + (content_height - title_font_->baseline()) / 2; // If the user has big fonts, the title will appear rendered too far down // on the y-axis if we use the regular top padding, so we need to adjust it |