chrome/browser/resources/managed_user_passphrase_dialog.html - Issue 11783008: Add a lock to the managed user settings page and require authentication for unlocking.

Side by Side Diff: chrome/browser/resources/managed_user_passphrase_dialog.html

Issue 11783008: Add a lock to the managed user settings page and require authentication for unlocking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 7 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 unified diff | Download patch | Annotate | Revision Log

« chrome/browser/resources/managed_user_passphrase_dialog.css ('K') | « chrome/browser/resources/managed_user_passphrase_dialog.css ('k') | chrome/browser/resources/managed_user_passphrase_dialog.js » ('j') | chrome/browser/resources/managed_user_passphrase_dialog.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE html>
	Pam (message me for reviews) 2013/01/07 14:51:49 Right after the DOCTYPE line: <!-- Copyright (c) Right after the DOCTYPE line: <!-- Copyright (c) 2012 The Chromium Authors. All rights reserved. Use of this source code is governed by a BSD-style license that can be found in the LICENSE file. --> Pam (message me for reviews) 2013/01/07 14:51:49 Right after the DOCTYPE line: <!-- Copyright (c) Right after the DOCTYPE line: <!-- Copyright (c) 2012 The Chromium Authors. All rights reserved. Use of this source code is governed by a BSD-style license that can be found in the LICENSE file. -->
	2 <html i18n-values="dir:textdirection">

	3 <head>

	4 <link rel="stylesheet" href="shared/css/chrome_shared.css">

	5 <link rel="stylesheet" href="dialog.css">

	6 <link rel="stylesheet" href="managed_user_passphrase_dialog.css">

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

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

	9 <script src="managed_user_passphrase_dialog.js"></script>

	10 </head>

	11 <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize">

	12 <div id="family-control-passphrase-page" class="page">
	Pam (message me for reviews) 2013/01/07 14:51:49 Please change "family" and "kid" to more general t Please change "family" and "kid" to more general terms (managed mode, managed user) to reflect our wider set of use cases.
	13 <img class="chrome_kid_logo_img" src="chrome_kid.png" alt="Chrome kid logo"
	Bernhard Bauer 2013/01/07 14:20:22 I don't think we need an alt attribute here. You c I don't think we need an alt attribute here. You could have a title (for accessibility), but we probably don't need it either, as the logo is purely decorative.
	14 width="65" height="65">
	Bernhard Bauer 2013/01/07 14:20:22 Width and height is probably also unnecessary? Rem Width and height is probably also unnecessary? Remember, we don't need to fetch this over the network, so having the layout in place before the image is loaded shouldn't be a big problem. Hm, thinking about it, do you think it would be possible to add the image only in CSS?
	15 <h1 i18n-content="familyControlPassphrasePage"></h1>

	16 <div class="content-area">

	17 <div id="instruct">

	18 <label>

	19 <span i18n-content="passphrase_instruction"></span>

	20 </label>

	21 </div>

	22 <div id="entry">

	23 <input id="passphrase_entry" size="50" type="password" />

	24 </div>

	25 <div id="ref">

	26 <a href="http://www.google.com" target="_blank">
	Pam (message me for reviews) 2013/01/07 14:51:49 Let's just comment out the whole "forgot passphras Let's just comment out the whole "forgot passphrase" div, until we have a place for it to point. (Probably a help page explaining the command-line recovery option, eventually.)
	27 <span i18n-content="forgot_passphrase"></span></a>

	28 </div>

	29 <div id="buttons">

	30 <div id="cancel">

	31 <button id="cancel_passphrase_button"

	32 i18n-content="cancel_passphrase_button"></button>

	33 <div id="unlock">

	34 <button id="unlock_passphrase_button"

	35 i18n-content="unlock_passphrase_button"></button>

	36 </div>

	37 </div>

	38 <div id="incorrect_passphrase_warning">

	39 <label>

	40 <span i18n-content="incorrect_passphrase_warning"></span>

	41 </label>

	42 </div>

	43 </div>

	44 </div>

	45 <script src="strings.js"></script>

	46 <script src="chrome://resources/js/i18n_template2.js"></script>
	Bernhard Bauer 2013/01/07 14:20:22 Can you move this to the header? Can you move this to the header?
	47 </body>

	48 </html>

OLD	NEW