Chromium Code Reviews Chromium Code Reviews
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| Index: chrome/browser/resources/managed_user_passphrase_dialog.html |
| diff --git a/chrome/browser/resources/managed_user_passphrase_dialog.html b/chrome/browser/resources/managed_user_passphrase_dialog.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..ccee7e271c0ff06ee16286db0d46a7a1c965638c |
| --- /dev/null |
| +++ b/chrome/browser/resources/managed_user_passphrase_dialog.html |
| @@ -0,0 +1,48 @@ |
| +<!DOCTYPE html> |
|
Pam (message me for reviews)
2013/01/07 14:51:49
Right after the DOCTYPE line:
<!-- Copyright (c)
Pam (message me for reviews)
2013/01/07 14:51:49
Right after the DOCTYPE line:
<!-- Copyright (c)
|
| +<html i18n-values="dir:textdirection"> |
| +<head> |
| + <link rel="stylesheet" href="shared/css/chrome_shared.css"> |
| + <link rel="stylesheet" href="dialog.css"> |
| + <link rel="stylesheet" href="managed_user_passphrase_dialog.css"> |
| + <script src="chrome://resources/js/load_time_data.js"></script> |
| + <script src="chrome://resources/js/util.js"></script> |
| + <script src="managed_user_passphrase_dialog.js"></script> |
| +</head> |
| +<body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize"> |
| + <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
|
| + <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
|
| + width="65" height="65"> |
|
Bernhard Bauer
2013/01/07 14:20:22
Width and height is probably also unnecessary? Rem
|
| + <h1 i18n-content="familyControlPassphrasePage"></h1> |
| + <div class="content-area"> |
| + <div id="instruct"> |
| + <label> |
| + <span i18n-content="passphrase_instruction"></span> |
| + </label> |
| + </div> |
| + <div id="entry"> |
| + <input id="passphrase_entry" size="50" type="password" /> |
| + </div> |
| + <div id="ref"> |
| + <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
|
| + <span i18n-content="forgot_passphrase"></span></a> |
| + </div> |
| + <div id="buttons"> |
| + <div id="cancel"> |
| + <button id="cancel_passphrase_button" |
| + i18n-content="cancel_passphrase_button"></button> |
| + <div id="unlock"> |
| + <button id="unlock_passphrase_button" |
| + i18n-content="unlock_passphrase_button"></button> |
| + </div> |
| + </div> |
| + <div id="incorrect_passphrase_warning"> |
| + <label> |
| + <span i18n-content="incorrect_passphrase_warning"></span> |
| + </label> |
| + </div> |
| + </div> |
| + </div> |
| + <script src="strings.js"></script> |
| + <script src="chrome://resources/js/i18n_template2.js"></script> |
|
Bernhard Bauer
2013/01/07 14:20:22
Can you move this to the header?
|
| +</body> |
| +</html> |
