Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(319)

Unified Diff: chrome/browser/ui/gtk/zoom_bubble_gtk.cc

Issue 10918275: [gtk] website settings bubble (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: slightly simpler drawing Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/gtk/website_settings/website_settings_popup_gtk.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/zoom_bubble_gtk.cc
diff --git a/chrome/browser/ui/gtk/zoom_bubble_gtk.cc b/chrome/browser/ui/gtk/zoom_bubble_gtk.cc
index dac8505a54921354215cc00604a566440c8cebec..c6dc5181a2d982432367f3f67861ad85ff6a4892 100644
--- a/chrome/browser/ui/gtk/zoom_bubble_gtk.cc
+++ b/chrome/browser/ui/gtk/zoom_bubble_gtk.cc
@@ -23,10 +23,6 @@ namespace {
// Pointer to singleton object (NULL if no bubble is open).
ZoomBubbleGtk* g_bubble = NULL;
-// Padding on each side of the percentage label and the left and right sides of
-// the "Set to default" button.
-const int kSidePadding = 5;
-
// Number of milliseconds the bubble should stay open for if it will auto-close.
const int kBubbleCloseDelay = 1500;
@@ -76,6 +72,7 @@ ZoomBubbleGtk::ZoomBubbleGtk(GtkWidget* anchor,
tab_contents->web_contents()->GetBrowserContext()));
event_box_ = gtk_event_box_new();
+ gtk_event_box_set_visible_window(GTK_EVENT_BOX(event_box_), FALSE);
GtkWidget* container = gtk_vbox_new(FALSE, 0);
gtk_container_add(GTK_CONTAINER(event_box_), container);
@@ -84,8 +81,8 @@ ZoomBubbleGtk::ZoomBubbleGtk(GtkWidget* anchor,
IDS_TOOLTIP_ZOOM, zoom_percent));
label_ = theme_service->BuildLabel(percentage_text, ui::kGdkBlack);
gtk_widget_modify_font(label_, pango_font_description_from_string("13"));
- gtk_misc_set_padding(GTK_MISC(label_), kSidePadding, kSidePadding);
-
+ gtk_misc_set_padding(GTK_MISC(label_),
+ ui::kControlSpacing, ui::kControlSpacing);
gtk_box_pack_start(GTK_BOX(container), label_, FALSE, FALSE, 0);
GtkWidget* separator = gtk_hseparator_new();
@@ -97,7 +94,7 @@ ZoomBubbleGtk::ZoomBubbleGtk(GtkWidget* anchor,
GtkWidget* alignment = gtk_alignment_new(0, 0, 1, 1);
gtk_alignment_set_padding(
- GTK_ALIGNMENT(alignment), 0, 0, kSidePadding, kSidePadding);
+ GTK_ALIGNMENT(alignment), 0, 0, ui::kControlSpacing, ui::kControlSpacing);
gtk_container_add(GTK_CONTAINER(alignment), set_default_button);
gtk_box_pack_start(GTK_BOX(container), alignment, FALSE, FALSE, 0);
« no previous file with comments | « chrome/browser/ui/gtk/website_settings/website_settings_popup_gtk.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698