chrome/browser/resources/sync_confirmation/sync_confirmation.html - Issue 1487283005: Implement the new Sync Confirmation dialog on Linux and Windows.

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

Issue 1487283005: Implement the new Sync Confirmation dialog on Linux and Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Add some tests. Created 5 years 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

« chrome/browser/resources/sync_confirmation/sync_confirmation.css ('K') | « chrome/browser/resources/sync_confirmation/sync_confirmation.css ('k') | chrome/browser/resources/sync_confirmation/sync_confirmation.js » ('j') | chrome/browser/ui/webui/signin/sync_confirmation_handler.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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

new file mode 100644

index 0000000000000000000000000000000000000000..d12444114a37703582268191d597e57a543311d6

--- /dev/null

+++ b/chrome/browser/resources/sync_confirmation/sync_confirmation.html

@@ -0,0 +1,35 @@

+<html>

Dan Beam 2016/01/06 22:57:57 <!doctype html> before <html>

anthonyvd 2016/01/08 22:39:41 Done.

+ <head>

Robert Sesek 2016/01/06 19:30:08 Does this page display internationalized content?

anthonyvd 2016/01/08 22:39:41 Done.

+ <link rel="stylesheet" type="text/css" href="sync-confirmation.css" media="screen"></link>

Dan Beam 2016/01/06 22:57:57 type="text/css" probably isn't necessary either

Dan Beam 2016/01/06 22:57:57 media="screen" isn't necessary unless you want to

anthonyvd 2016/01/08 22:39:41 Done.

+ <script src="chrome://resources/js/cr.js"></script>

+ <script src="chrome://resources/js/load_time_data.js"></script>

+ <script src="chrome://resources/js/util.js"></script>

+ <script type="text/javascript" src="sync-confirmation.js"></script>

Dan Beam 2016/01/06 22:57:57 remove type="text/javascript"

anthonyvd 2016/01/08 22:39:41 Done.

+ </head>

+ <body>

+ <div class="container">

+ <div class="top-title-bar" i18n-content="syncConfirmationTitle"></div>

+ <div class="details">

+ <div class="picture-container">

+ <div class="picture">

+ <img id="profile-picture"></img>

+ <div class="checkmark-bubble"></div>

+ </div>

+ <div class="sync-message" i18n-values=".innerHTML:syncConfirmationBody"></div>

+ <div class="action-container">

+ <div class="action-button-wrapper">

+ <div class="action-button" id="confirmButton">

+ <span class="action-button-label" i18n-content="syncConfirmationConfirmLabel"></span>

+ </div>

+ <div class="link-wrapper">

+ <span class="link" id="undoLink" i18n-content="syncConfirmationUndoLabel"></span>

+ </div>

+ </body>

+ <script src="chrome://sync-confirmation/strings.js"></script>

+ <script src="chrome://resources/js/i18n_template.js"></script>

+</html>