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

Unified Diff: chrome/browser/resources/sync_confirmation/sync_confirmation.html

Issue 1868273002: Enhanced Sync Confirmation modal (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/resources/sync_confirmation/sync_confirmation.html
diff --git a/chrome/browser/resources/sync_confirmation/sync_confirmation.html b/chrome/browser/resources/sync_confirmation/sync_confirmation.html
index 5d287a5a7d55afe2d7bccb5e624c93707d6dc332..a783011debbc06c83e97d295d7c3e9bbaca0b73f 100644
--- a/chrome/browser/resources/sync_confirmation/sync_confirmation.html
+++ b/chrome/browser/resources/sync_confirmation/sync_confirmation.html
@@ -6,6 +6,13 @@
<link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
<link rel="stylesheet" href="chrome://resources/css/text_defaults_md.css">
<link rel="stylesheet" href="sync_confirmation.css"></link>
+ <!-- see https://www.polymer-project.org/1.0/docs/devguide/styling.html#custom-style -->
Dan Beam 2016/04/14 04:05:56 i don't think you need this note
Moe 2016/04/14 14:26:53 Done.
+ <style is="custom-style">
+ a {
+ color: var(--google-blue-700);
+ text-decoration: none;
+ }
+ </style>
</head>
<body>
<div class="container">
@@ -38,14 +45,35 @@
</div>
</div>
</div>
- <div class="sync-message"
- i18n-values=".innerHTML:syncConfirmationChromeSyncBody"></div>
- <div class="action-container">
- <paper-button id="confirmButton"
- i18n-content="syncConfirmationConfirmLabel"></paper-button>
- <paper-button id="undoButton"
- i18n-content="syncConfirmationUndoLabel"></paper-button>
+ <div class="message-container">
+ <div id="chrome-logo" class="logo"></div>
+ <div>
+ <div class="title"
+ i18n-content="syncConfirmationChromeSyncTitle"></div>
Dan Beam 2016/04/14 04:05:56 nit: <div class="title">$i18n{syncConfirmationC
Moe 2016/04/14 14:26:53 Cool! Done.
+ <div class="body text"
+ i18n-values=".innerHTML:syncConfirmationChromeSyncBody"></div>
+ </div>
+ </div>
+ <div class="message-container">
+ <div id="googleg-logo" class="logo"></div>
+ <div>
+ <div class="title"
+ i18n-content="syncConfirmationPersonalizeServicesTitle"></div>
+ <div class="body text"
+ i18n-content="syncConfirmationPersonalizeServicesBody"></div>
+ </div>
</div>
+ <div class="message-container">
+ <div class="body"
+ i18n-values=".innerHTML:syncConfirmationSyncSettingsLinkBody">
+ </div>
+ </div>
+ </div>
+ <div class="action-container">
+ <paper-button id="confirmButton"
+ i18n-content="syncConfirmationConfirmLabel"></paper-button>
+ <paper-button id="undoButton"
+ i18n-content="syncConfirmationUndoLabel"></paper-button>
</div>
</div>
</body>

Powered by Google App Engine
This is Rietveld 408576698