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

Unified Diff: chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm

Issue 11077006: Correct padding and focus ring for frameless constrained window dialog (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix license header Created 8 years, 2 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: chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm
diff --git a/chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm b/chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm
index 569e2e86f4039389e271aed6322a61d081251266..aaf57d7010effa1d8641bf58644f7d1600d685cc 100644
--- a/chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm
+++ b/chrome/browser/ui/cocoa/intents/web_intent_message_view_controller.mm
@@ -7,6 +7,7 @@
#import "chrome/browser/ui/cocoa/constrained_window/constrained_window_control_utils.h"
#import "chrome/browser/ui/cocoa/flipped_view.h"
#import "chrome/browser/ui/constrained_window.h"
+#import "chrome/browser/ui/constrained_window_constants.h"
#include "chrome/browser/ui/intents/web_intent_picker.h"
#include "third_party/GTM/AppKit/GTMUILocalizerAndLayoutTweaker.h"
@@ -35,7 +36,7 @@
[titleTextField_ setAttributedStringValue:
constrained_window::GetAttributedLabelString(
title,
- ConstrainedWindow::kTitleFontStyle,
+ ConstrainedWindowConstants::kTitleFontStyle,
NSNaturalTextAlignment,
NSLineBreakByWordWrapping)];
}
@@ -44,7 +45,7 @@
[messageTextField_ setAttributedStringValue:
constrained_window::GetAttributedLabelString(
message,
- ConstrainedWindow::kTextFontStyle,
+ ConstrainedWindowConstants::kTextFontStyle,
NSNaturalTextAlignment,
NSLineBreakByWordWrapping)];
}
@@ -53,7 +54,7 @@
[self resizeTextFieldsToWidth:innerWidth];
CGFloat height = NSHeight([titleTextField_ frame]);
height += NSHeight([messageTextField_ frame]);
- height += ConstrainedWindow::kRowPadding;
+ height += ConstrainedWindowConstants::kRowPadding;
return NSMakeSize(innerWidth, height);
}
@@ -66,7 +67,8 @@
NSRect messageFrame = [messageTextField_ frame];
messageFrame.origin.x = NSMinX(innerFrame);
- messageFrame.origin.y = NSMaxY(titleFrame) + ConstrainedWindow::kRowPadding;
+ messageFrame.origin.y = NSMaxY(titleFrame) +
+ ConstrainedWindowConstants::kRowPadding;
[messageTextField_ setFrame:messageFrame];
}

Powered by Google App Engine
This is Rietveld 408576698