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

Unified Diff: ui/views/controls/button/label_button.h

Issue 872113004: Modified OverviewMode's LabelButton bounds to cover the entire item. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unnecessary changes from unittests Created 5 years, 11 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
Index: ui/views/controls/button/label_button.h
diff --git a/ui/views/controls/button/label_button.h b/ui/views/controls/button/label_button.h
index 2efd5b11f7d273e19eba4cdc95bbac37e5f4eee5..11bdb667e11020781d74c05c97cf277e279485d2 100644
--- a/ui/views/controls/button/label_button.h
+++ b/ui/views/controls/button/label_button.h
@@ -85,6 +85,12 @@ class VIEWS_EXPORT LabelButton : public CustomButton,
void SetFocusPainter(scoped_ptr<Painter> focus_painter);
Painter* focus_painter() { return focus_painter_.get(); }
+ // Set whether the label should ignore the vertical overlap with the border.
+ // Default is true.
+ void set_ignore_vertical_overlap(bool ignore_vertical_overlap) {
+ ignore_vertical_overlap_ = ignore_vertical_overlap;
+ }
+
// View:
void SetBorder(scoped_ptr<Border> border) override;
gfx::Size GetPreferredSize() const override;
@@ -191,6 +197,9 @@ class VIEWS_EXPORT LabelButton : public CustomButton,
// Spacing between the image and the text.
int image_label_spacing_;
+ // Indicates if the label should ignore the vertical overlap with the border.
+ bool ignore_vertical_overlap_;
+
scoped_ptr<Painter> focus_painter_;
DISALLOW_COPY_AND_ASSIGN(LabelButton);

Powered by Google App Engine
This is Rietveld 408576698