 Chromium Code Reviews
 Chromium Code Reviews Issue 6034005:
  DOMUI: Implement the new-style Autofill options page.  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src
    
  
    Issue 6034005:
  DOMUI: Implement the new-style Autofill options page.  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: chrome/browser/resources/options/autofill_options.html | 
| diff --git a/chrome/browser/resources/options/autofill_options.html b/chrome/browser/resources/options/autofill_options.html | 
| index c5bbb72de63441240aefd917f4781491f72b777d..430322b440a3d86362dc45e6b53fb25e1a587ef7 100644 | 
| --- a/chrome/browser/resources/options/autofill_options.html | 
| +++ b/chrome/browser/resources/options/autofill_options.html | 
| @@ -1,38 +1,26 @@ | 
| <div id="autofill-options" class="page hidden"> | 
| <h1 i18n-content="autoFillOptionsTitle"></h1> | 
| - | 
| - <div id="profile-select-container" class="autofill-section"> | 
| - <div id="profile-select-left-column"> | 
| - <select id="profile-list" size="10"> | 
| - <option value="addressesHeader" disabled | 
| - i18n-content="addressesHeader"></option> | 
| - <option value="addressesHorizontalRule" disabled></option> | 
| - <option value="addressesBlankOption" disabled></option> | 
| - <option value="creditCardsHeader" disabled | 
| - i18n-content="creditCardsHeader"></option> | 
| - <option value="creditCardsHorizontalRule" disabled></option> | 
| - </select> | 
| - </div> | 
| - <div> | 
| + <div> | 
| + <div class="list-title" i18n-content="autoFillAddresses"></div> | 
| + <div class="settings-list"> | 
| + <list id="address-list"></list> | 
| <div> | 
| 
arv (Not doing code reviews)
2010/12/22 18:14:07
Can this div be skipped?
 
James Hawkins
2010/12/22 21:22:31
No, it's the lower-button div which has a border l
 | 
| - <button id="add-address-button" i18n-content="addAddressButton"> | 
| + <button id="autofill-add-address" i18n-content="autoFillAddAddress"> | 
| </button> | 
| </div> | 
| + </div> | 
| + </div> | 
| + <div> | 
| + <div class="list-title" i18n-content="autoFillCreditCards"></div> | 
| + <div class="settings-list"> | 
| + <list id="creditcard-list"></list> | 
| <div> | 
| 
arv (Not doing code reviews)
2010/12/22 18:14:07
same here?
 
James Hawkins
2010/12/22 21:22:31
Same as above.
 | 
| - <button id="add-credit-card-button" i18n-content="addCreditCardButton"> | 
| - </button> | 
| - </div> | 
| - <div> | 
| - <button id="autofill-edit-button" i18n-content="editButton"></button> | 
| - </div> | 
| - <div> | 
| - <button id="autofill-remove-button" i18n-content="deleteButton"> | 
| - </button> | 
| + <button id="autofill-add-creditcard" | 
| + i18n-content="autoFillAddCreditCard"></button> | 
| </div> | 
| </div> | 
| </div> | 
| - | 
| - <div class="autofill-section"> | 
| + <div> | 
| <if expr="pp_ifdef('chromeos')"> | 
| <a href="http://www.google.com/support/chromeos/bin/answer.py?answer=142893" | 
| target="_blank" i18n-content="helpButton"></a> |