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

Unified Diff: ui/views/bubble/bubble_frame_view.cc

Issue 2610613002: Ensure the Harmony close buttons appear in the correct place. (Closed)
Patch Set: Harmony -> MD Created 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/bubble/bubble_frame_view.cc
diff --git a/ui/views/bubble/bubble_frame_view.cc b/ui/views/bubble/bubble_frame_view.cc
index 7cc53e0bc1e4db158a15132ef59331db1167399d..c22d511771c21b1b10e67ae4e89b91b281ee2adc 100644
--- a/ui/views/bubble/bubble_frame_view.cc
+++ b/ui/views/bubble/bubble_frame_view.cc
@@ -45,6 +45,14 @@ const SkColor kFootnoteBackgroundColor = SkColorSetRGB(245, 245, 245);
// Color of the top border of the footnote.
const SkColor kFootnoteBorderColor = SkColorSetRGB(229, 229, 229);
+constexpr int kClosePaddingRight = 7;
+constexpr int kClosePaddingTop = 6;
+
+// The MD spec states that the center of the "x" should be 16x16 from the top
+// right of the dialog.
+constexpr int kClosePaddingRightMd = 4;
+constexpr int kClosePaddingTopMd = 5;
+
// Get the |vertical| or horizontal amount that |available_bounds| overflows
// |window_bounds|.
int GetOffScreenLength(const gfx::Rect& available_bounds,
@@ -314,7 +322,13 @@ void BubbleFrameView::Layout() {
// The close button is positioned somewhat closer to the edge of the bubble.
gfx::Point close_position = GetContentsBounds().top_right();
- close_position += gfx::Vector2d(-close_->width() - 7, 6);
+ if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
+ close_position += gfx::Vector2d(-close_->width() - kClosePaddingRightMd,
+ kClosePaddingTopMd);
+ } else {
+ close_position +=
+ gfx::Vector2d(-close_->width() - kClosePaddingRight, kClosePaddingTop);
+ }
close_->SetPosition(close_position);
gfx::Size title_icon_pref_size(title_icon_->GetPreferredSize());
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698