Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(83)

Side by Side Diff: chrome/browser/resources/options/autofill_options.html

Issue 1583293004: Changes to Autofill settings page to ensure Credit Card list is always at least partially visible o… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Spacing tweak Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 <div id="autofill-options" class="page" hidden> 1 <div id="autofill-options" class="page" hidden>
2 <div class="close-button"></div> 2 <div class="close-button"></div>
3 <h1 i18n-content="autofillOptionsPage"></h1> 3 <h1 i18n-content="autofillOptionsPage"></h1>
4 <div class="content-area"> 4 <div class="content-area">
5 <button id="autofill-add-address" i18n-content="autofillAddAddress">
6 </button>
5 <h3 i18n-content="autofillAddresses"></h3> 7 <h3 i18n-content="autofillAddresses"></h3>
6 <div class="settings-list"> 8 <div class="settings-list">
7 <list id="address-list"></list> 9 <list id="address-list"></list>
8 <div>
9 <button id="autofill-add-address" i18n-content="autofillAddAddress">
10 </button>
11 </div>
12 </div> 10 </div>
13 11 <div class="vertical-space"></div>
12 <button id="autofill-add-creditcard" i18n-content="autofillAddCreditCard">
13 </button>
14 <h3 i18n-content="autofillCreditCards"></h3> 14 <h3 i18n-content="autofillCreditCards"></h3>
15 <div class="settings-list"> 15 <div class="settings-list">
16 <list id="creditcard-list"></list> 16 <list id="creditcard-list"></list>
17 <div>
18 <button id="autofill-add-creditcard"
19 i18n-content="autofillAddCreditCard"></button>
20 </div>
21 </div> 17 </div>
22 </div> 18 </div>
23 19
24 <div class="action-area"> 20 <div class="action-area">
25 <a id="autofill-help" target="_blank" i18n-values="href:helpUrl" 21 <a id="autofill-help" target="_blank" i18n-values="href:helpUrl"
26 i18n-content="helpButton"> 22 i18n-content="helpButton">
27 </a> 23 </a>
28 <div class="spacer-div"></div> 24 <div class="spacer-div"></div>
Mathieu 2016/01/18 20:07:36 I notice we have class spacer-div, but I can't see
tmartino 2016/01/18 21:05:21 I tracked it down to somewhere murky in the WebUI
29 <div class="button-strip"> 25 <div class="button-strip">
30 <button id="autofill-options-confirm" class="default-button" 26 <button id="autofill-options-confirm" class="default-button"
31 i18n-content="done"></button> 27 i18n-content="done"></button>
32 </div> 28 </div>
33 </div> 29 </div>
34 </div> 30 </div>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698