Chromium Code Reviews| Index: chrome/browser/resources/signin/signin_email_confirmation/signin_email_confirmation.html |
| diff --git a/chrome/browser/resources/signin/signin_email_confirmation/signin_email_confirmation.html b/chrome/browser/resources/signin/signin_email_confirmation/signin_email_confirmation.html |
| index c81130284be54ce4cdfb5aa0215e3d92b69421c8..b3afbc0c508428e8a65faf7034a5ea2b37672b3d 100644 |
| --- a/chrome/browser/resources/signin/signin_email_confirmation/signin_email_confirmation.html |
| +++ b/chrome/browser/resources/signin/signin_email_confirmation/signin_email_confirmation.html |
| @@ -15,9 +15,31 @@ |
| <style is="custom-style" include="signin-dialog-shared"> |
| .details { |
| line-height: 20px; |
| - margin-bottom: 8px; |
| - margin-left: -8px; |
| - padding: 0 24px; |
| + padding: 8px 24px 0; |
| + -webkit-margin-start: -8px; |
|
Dan Beam
2017/01/12 18:32:58
this should go at the top of this rule
msarda
2017/01/13 09:54:16
Done.
|
| + } |
| + |
| + .container { |
| + /* The sign-in confirmation dialog uses a width that is larger than |
| + * the other sign-in dialogs to fit cases when the user has an email |
| + * name that is very large. */ |
| + width: 512px; |
| + } |
| + |
| + .radio-button-title-container { |
| + line-height: 20px; |
| + } |
| + |
| + .top-title-bar { |
| + color: #333; |
| + |
| + /* The signin email confirmation dialog needs to support cases when the |
| + * title extends multiple lines. In that case, the height of the title |
| + * bar is equal to padding + height of the text (which is equal to |
| + * 2*16px + nb_lines * 22px). */ |
| + line-height: 22px; |
| + padding: 16px 24px; |
| + height: auto; |
|
Dan Beam
2017/01/12 18:32:58
please alphabetize these rules
msarda
2017/01/13 09:54:16
Done (I also alphabetize the classes).
|
| } |
| #closeButton { |
| @@ -41,10 +63,16 @@ |
| <paper-radio-group selected="createNewUser"> |
| <paper-radio-button id="createNewUserRadioButton" |
| name="createNewUser"> |
| - $i18n{signinEmailConfirmationCreateProfileButtonTitle} |
| + <div class="radio-button-title-container" |
| + id="createNewUserRadioButtonTitle"> |
| + $i18n{signinEmailConfirmationCreateProfileButtonTitle} |
| + </div> |
| </paper-radio-button> |
| <paper-radio-button id="startSyncRadioButton" name="startSync"> |
| - $i18n{signinEmailConfirmationStartSyncButtonTitle} |
| + <div class="radio-button-title-container" |
| + id="startSyncRadioButtonTitle"> |
| + $i18n{signinEmailConfirmationStartSyncButtonTitle} |
| + </div> |
| </paper-radio-button> |
| </paper-radio-group> |
| </div> |