Chromium Code Reviews| Index: chrome/browser/resources/options/password_manager.html |
| =================================================================== |
| --- chrome/browser/resources/options/password_manager.html (revision 86208) |
| +++ chrome/browser/resources/options/password_manager.html (working copy) |
| @@ -1,6 +1,13 @@ |
| <div id="password-manager" class="page hidden"> |
| <h1 i18n-content="passwordsPage"></h1> |
| - <h3 i18n-content="savedPasswordsTitle"></h3> |
| + <div id="password-list-headers"> |
| + <div id="passwords-title"><h3 i18n-content="savedPasswordsTitle"></h3></div> |
|
James Hawkins
2011/05/22 00:02:58
For readability, can you put the <h3> block on a n
Mike Mammarella
2011/05/22 00:35:11
Done.
|
| + <div id="password-search-column"> |
| + <input id="password-search-box" type="search" |
| + placeholder=" Search passwords" incremental |
|
James Hawkins
2011/05/22 00:02:58
What's the space at the beginning of the placehold
Mike Mammarella
2011/05/22 00:35:11
Makes it look nicer. There's a magnifying glass ic
James Hawkins
2011/05/22 02:03:46
Yes, please; very important for i18n. Do we do the
|
| + results="10" autosave="org.chromium.options.passwords.search"> |
| + </div> |
| + </div> |
| <list id="saved-passwords-list" class="settings-list"></list> |
| <div id="saved-passwords-list-empty-placeholder" |
| class="settings-list-empty" hidden> |