chrome/browser/resources/options/autofill_options.html - Issue 6034005: DOMUI: Implement the new-style Autofill options page.

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

Issue 6034005: DOMUI: Implement the new-style Autofill options page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Review fixes. Created 10 years 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 | Annotate | Revision Log

« chrome/browser/resources/options/autofill_options.css ('K') | « chrome/browser/resources/options/autofill_options.css ('k') | chrome/browser/resources/options/autofill_options.js » ('j') | chrome/browser/resources/options/autofill_options_list.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <div id="autofill-options" class="page hidden">	1 <div id="autofill-options" class="page hidden">

2 <h1 i18n-content="autoFillOptionsTitle"></h1>	2 <h1 i18n-content="autoFillOptionsTitle"></h1>

3	3 <div>

4 <div id="profile-select-container" class="autofill-section">	4 <div class="list-title" i18n-content="autoFillAddresses"></div>

5 <div id="profile-select-left-column">	5 <div class="settings-list">

6 <select id="profile-list" size="10">	6 <list id="address-list"></list>

7 <option value="addressesHeader" disabled

8 i18n-content="addressesHeader"></option>

9 <option value="addressesHorizontalRule" disabled></option>

10 <option value="addressesBlankOption" disabled></option>

11 <option value="creditCardsHeader" disabled

12 i18n-content="creditCardsHeader"></option>

13 <option value="creditCardsHorizontalRule" disabled></option>

14 </select>

15 </div>

16 <div>

17 <div>	7 <div>
	arv (Not doing code reviews) 2010/12/22 18:14:07 Can this div be skipped? 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 Show quoted text On 2010/12/22 18:14:07, arv wrote: > Can this div be skipped? No, it's the lower-button div which has a border line at the top.
18 <button id="add-address-button" i18n-content="addAddressButton">	8 <button id="autofill-add-address" i18n-content="autoFillAddAddress">

19 </button>

20 </div>

21 <div>

22 <button id="add-credit-card-button" i18n-content="addCreditCardButton">

23 </button>

24 </div>

25 <div>

26 <button id="autofill-edit-button" i18n-content="editButton"></button>

27 </div>

28 <div>

29 <button id="autofill-remove-button" i18n-content="deleteButton">

30 </button>	9 </button>

31 </div>	10 </div>

32 </div>	11 </div>

33 </div>	12 </div>

34	13 <div>

35 <div class="autofill-section">	14 <div class="list-title" i18n-content="autoFillCreditCards"></div>

	15 <div class="settings-list">

	16 <list id="creditcard-list"></list>

	17 <div>
	arv (Not doing code reviews) 2010/12/22 18:14:07 same here? same here? James Hawkins 2010/12/22 21:22:31 Same as above. Show quoted text On 2010/12/22 18:14:07, arv wrote: > same here? Same as above.
	18 <button id="autofill-add-creditcard"

	19 i18n-content="autoFillAddCreditCard"></button>

	20 </div>

	21 </div>

	22 </div>

	23 <div>

36 <if expr="pp_ifdef('chromeos')">	24 <if expr="pp_ifdef('chromeos')">

37 <a href="http://www.google.com/support/chromeos/bin/answer.py?answer=142893"	25 <a href="http://www.google.com/support/chromeos/bin/answer.py?answer=142893"

38 target="_blank" i18n-content="helpButton"></a>	26 target="_blank" i18n-content="helpButton"></a>

39 </if>	27 </if>

40 <if expr="not pp_ifdef('chromeos')">	28 <if expr="not pp_ifdef('chromeos')">

41 <a href="http://www.google.com/support/chrome/bin/answer.py?answer=142893"	29 <a href="http://www.google.com/support/chrome/bin/answer.py?answer=142893"

42 target="_blank" i18n-content="helpButton"></a>	30 target="_blank" i18n-content="helpButton"></a>

43 </if>	31 </if>

44 </div>	32 </div>

45 </div>	33 </div>

OLD	NEW