Chromium Code Reviews Chromium Code Reviews
Issue 7058048:
[cros] Layout for OOBE WebUI. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: chrome/browser/resources/chromeos/oobe.html |
| diff --git a/chrome/browser/resources/chromeos/oobe.html b/chrome/browser/resources/chromeos/oobe.html |
| index 4c555689b06bdc4327c1ada26cfee009d4e6a856..fe6abbf93c43002c7175af3f4967fa214399f4af 100644 |
| --- a/chrome/browser/resources/chromeos/oobe.html |
| +++ b/chrome/browser/resources/chromeos/oobe.html |
| @@ -2,8 +2,88 @@ |
| <html i18n-values="dir:textdirection"> |
| <head> |
| <title i18n-content="title"></title> |
| +<link rel="stylesheet" href="chrome://resources/css/button.css"> |
| +<link rel="stylesheet" href="chrome://resources/css/select.css"> |
| <link rel="stylesheet" href="oobe.css"> |
| +<script src="chrome://resources/js/cr.js"></script> |
| +<script src="chrome://resources/js/cr/event_target.js"></script> |
| +<script src="chrome://resources/js/local_strings.js"></script> |
| +<script src="chrome://resources/js/util.js"></script> |
| +<script src="oobe.js"></script> |
| </head> |
| <body i18n-values=".style.fontFamily:fontfamily;"> |
| + <div id="oobe" class="connect"> |
| + <div id="outer-container"> |
| + <div id="header"> |
| + <span i18n-content="productName"></span> |
| + <div id="header-sections"> |
| + <span id="hconnect" class="header-section" i18n-content="welcomeScreenTitle"></span> |
|
Evan Stade
2011/06/07 03:25:46
don't abbreviate for variable names (hconnect->hea
Nikita (slow)
2011/06/07 17:03:36
It's used in JS:
var oldheader = $("h" + steps[thi
Nikita (slow)
2011/06/07 17:03:36
Done.
|
| + <span id="heula" class="header-section right" i18n-content="eulaScreenTitle"></span> |
| + <span id="hupdate" class="header-section right" i18n-content="updateScreenTitle"></span> |
|
Evan Stade
2011/06/07 03:25:46
80 still applies even in html and css files
Nikita (slow)
2011/06/07 17:03:36
Done.
|
| + </div> |
| + </div> |
| + <hr class="topshadow"> |
| + <div id="inner-container"> |
| + <div class="step" id="connect"> |
| + <table> |
|
Evan Stade
2011/06/07 03:25:46
I'm skeptical you really want a table, this doesn'
Nikita (slow)
2011/06/07 17:03:36
I've using this layout mostly as-is done by the de
|
| + <tr height='40px'> |
| + <td width='200px' i18n-content="languageSelect"></td> |
| + <td> |
| + <select id='language-select'></select> |
| + </td> |
| + </tr> |
| + <tr height='40px'> |
| + <td i18n-content="keyboardSelect"></td> |
| + <td> |
| + <select id='keyboard-select'></select> |
| + </td> |
| + </tr> |
| + <tr height='40px'> |
|
Evan Stade
2011/06/07 03:25:46
do not use height attribute
Nikita (slow)
2011/06/07 17:03:36
At all or move it to css style?
Evan Stade
2011/06/08 19:56:07
move to css
Nikita (slow)
2011/06/08 22:28:27
Not needed any more.
|
| + <td i18n-content="networkSelect"></td> |
| + <td> |
| + <!-- TODO(nkostylev): Embed network control in a separate window. --> |
| + </td> |
| + </tr> |
| + </table> |
| + </div> |
| + <div class="step right hidden" id="eula"> |
| + <div id="eulas" class="eula-columns one-column"> |
| + <div id="cros-eula"> |
| + <iframe id="cros-eula-frame" class="eula-frame" src="chrome://about/terms"></iframe> |
| + </div> |
| + <div id="oem-eula"><iframe id="oem-eula-frame" class="eula-frame"></iframe></div> |
| + </div> |
| + <p id="logging"> |
| + <input type="checkbox"/><span i18n-content="checkboxLogging"></span> |
| + <a id="stats-help-link" href="#" i18n-content="learnMore"></a> |
| + </p> |
| + </div> |
| + <div class="step right hidden" id="update"> |
| + <div> |
| + <p i18n-content="installingUpdate"></p> |
| + <progress id="" max="100" value="0"></progress> |
| + <p i18n-content="installingUpdateDesc"></p> |
| + </div> |
| + </div> |
| + </div> <!-- Inner Container --> |
| + <hr class="bottomshadow"> |
| + <div id="footer"> |
| + <div id="progress"> |
| + <div id="connect-dot" class="progdot"></div> |
| + <div id="eula-dot" class="progdot"></div> |
| + <div id="signin-dot" class="progdot"></div> |
| + <div id="photo-dot" class="progdot"></div> |
| + </div> |
| + <div class="button-strip"> |
| + <button id="continue-button" i18n-content="continue"></button> |
| + <button id="back-button" i18n-content="back"></button> |
| + <button id="accept-button" i18n-content="acceptAgreement"></button> |
| + </div> |
| + </div> <!-- Footer --> |
| + </div> <!-- Outer Container --> |
| + </div> <!-- OOBE --> |
| + <div id="tpm"> |
| + <a id="tpm-link" href="#" i18n-content="tpmSetting"></a> |
| + </div> |
| </body> |
| </html> |
