Chromium Code Reviews Chromium Code Reviews
Issue 16104008:
First try at a user management screen for the desktop (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/browser/resources/user_chooser/user_chooser.html |
| diff --git a/chrome/browser/resources/user_chooser/user_chooser.html b/chrome/browser/resources/user_chooser/user_chooser.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..94cfe72965a3016fb3fbf651b2db0218ed73b60f |
| --- /dev/null |
| +++ b/chrome/browser/resources/user_chooser/user_chooser.html |
| @@ -0,0 +1,50 @@ |
| +<!DOCTYPE HTML> |
| +<html i18n-values="dir:textdirection;screen:screenType;build:buildType"> |
| +<head> |
| +<title i18n-content="title"></title> |
| +<link rel="stylesheet" href="../chromeos/login/oobe.css"> |
| +<link rel="stylesheet" href="../chromeos/login/user_pod_row.css"> |
| +<link rel="stylesheet" href="../chromeos/login/header_bar.css"> |
| +<!-- desktop user chooser specific overrides --> |
| +<link rel="stylesheet" href="control_bar.css"> |
| +<link rel="stylesheet" href="../chromeos/login/bubble.css"> |
| +<link rel="stylesheet" href="../chromeos/login/screen_account_picker.css"> |
| +<link rel="stylesheet" href="../chromeos/login/screen_container.css"> |
| +<!-- framework imports --> |
| +<!-- as per chrome/browser/resources/chromeos/login/login_resources.html --> |
| +<script src="chrome://resources/js/cr.js"></script> |
| +<script src="chrome://resources/js/event_tracker.js"></script> |
| +<script src="chrome://resources/js/cr/event_target.js"></script> |
| +<script src="chrome://resources/js/cr/ui.js"></script> |
| +<script src="chrome://resources/js/cr/ui/array_data_model.js"></script> |
| +<script src="chrome://resources/js/cr/ui/list_selection_controller.js"></script> |
| +<script src="chrome://resources/js/cr/ui/list_selection_model.js"></script> |
| +<script src="chrome://resources/js/cr/ui/list_single_selection_model.js"></script> |
| +<script src="chrome://resources/js/cr/ui/list.js"></script> |
| +<script src="chrome://resources/js/cr/ui/list_item.js"></script> |
| +<script src="chrome://resources/js/cr/ui/grid.js"></script> |
| +<script src="chrome://resources/js/cr/ui/menu_item.js"></script> |
| +<script src="chrome://resources/js/cr/ui/menu.js"></script> |
| +<script src="chrome://resources/js/cr/ui/menu_button.js"></script> |
| +<script src="chrome://resources/js/load_time_data.js"></script> |
| +<script src="chrome://resources/js/util.js"></script> |
| +<script src="user_chooser.js"></script> |
| +<script src="chrome://user-chooser/strings.js"></script> |
| +</head> |
| +<body class="login-display" i18n-values=".style.fontFamily:fontfamily;"> |
| +<div id="outer-container"> |
| + <div id="oobe" class="faded"> |
| + <div id="inner-container" class="down"> |
|
Nikita (slow)
2013/06/14 16:51:35
nit: 2 space indentation
Nikita (slow)
2013/06/14 16:51:35
nit: drop class="down" attribute, only needed on C
noms (inactive)
2013/06/28 17:31:04
Done.
noms (inactive)
2013/06/28 17:31:04
Done.
|
| + <div id="step-logo" hidden> |
| + <div id="header-sections"></div> |
| + </div> |
|
Nikita (slow)
2013/06/14 16:51:35
nit: align with opening div tag
noms (inactive)
2013/06/28 17:31:04
Done.
|
| + <include src="../chromeos/login/screen_account_picker.html"> |
| + </div> |
|
Nikita (slow)
2013/06/14 16:51:35
nit: align with opening div tag
noms (inactive)
2013/06/28 17:31:04
Done.
|
| + </div> |
| +</div> |
| +<div id="bubble" class="bubble faded" hidden></div> |
|
Nikita (slow)
2013/06/14 16:51:35
nit: 2 space indentation
noms (inactive)
2013/06/28 17:31:04
Hmm, does this need to be indented, given that it'
Nikita (slow)
2013/07/02 14:20:12
Yes, it should. In fact everything inside <body> (
|
| +<include src="control_bar.html"> |
| +<include src="desktop_user_pod_template.html"> |
| +<script src="chrome://resources/js/i18n_template2.js"></script> |
| +</body> |
| +</html> |
