chrome/browser/resources/options/autofill_edit_address_overlay.html - Issue 7892048: Autofill: Remove fax number completely.

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

Issue 7892048: Autofill: Remove fax number completely. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 9 years, 3 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 | Annotate | Revision Log

« chrome/browser/autofill/phone_number.h ('K') | « chrome/browser/automation/testing_automation_provider.cc ('k') | chrome/browser/resources/options/autofill_edit_address_overlay.js » ('j') | chrome/browser/resources/options/autofill_edit_address_overlay.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <div id="autofill-edit-address-overlay" class="page" hidden>	1 <div id="autofill-edit-address-overlay" class="page" hidden>

2 <h1 id="autofill-address-title"></h1>	2 <h1 id="autofill-address-title"></h1>

3 <div class="content-area">	3 <div class="content-area">

4 <div class="input">	4 <div class="input">

5 <label>	5 <label>

6 <div><span i18n-content="fullNameLabel"></span></div>	6 <div><span i18n-content="fullNameLabel"></span></div>

7 <list id="full-name-list"	7 <list id="full-name-list"

8 i18n-values="placeholder:addNewNamePlaceholder"></list>	8 i18n-values="placeholder:addNewNamePlaceholder"></list>

9 </label>	9 </label>

10 </div>	10 </div>

(...skipping 49 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
60 <select id="country"></select>	60 <select id="country"></select>

61 </label>	61 </label>

62 </div>	62 </div>

63 <div class="table">	63 <div class="table">

64 <div class="row">	64 <div class="row">

65 <div class="input cell">	65 <div class="input cell">

66 <label for="phone-list">	66 <label for="phone-list">

67 <span i18n-content="phoneLabel"></span>	67 <span i18n-content="phoneLabel"></span>

68 </label>	68 </label>

69 </div>	69 </div>

70 <div class="input cell">

71 <label for="fax-list">

72 <span i18n-content="faxLabel"></span>

73 </label>

74 </div>

75 </div>	70 </div>

76 <div class="row">	71 <div class="row">

77 <div class="input cell">	72 <div class="input cell">

78 <list id="phone-list"	73 <list id="phone-list"

79 i18n-values="placeholder:addNewPhonePlaceholder"></list>	74 i18n-values="placeholder:addNewPhonePlaceholder"></list>

80 </div>	75 </div>

81 <div class="input cell">

82 <list id="fax-list"

83 i18n-values="placeholder:addNewFaxPlaceholder"></list>

84 </div>

85 </div>	76 </div>

86 </div>	77 </div>

87 <div class="table">	78 <div class="table">

88 <div class="row">	79 <div class="row">

89 <div class="input cell">	80 <div class="input cell">

90 <label for="email-list">	81 <label for="email-list">

91 <span i18n-content="emailLabel"></span>	82 <span i18n-content="emailLabel"></span>

92 </label>	83 </label>

93 </div>	84 </div>

94 </div>	85 </div>

95 <div class="row">	86 <div class="row">

96 <div class="input cell">	87 <div class="input cell">

97 <list id="email-list"	88 <list id="email-list"

98 i18n-values="placeholder:addNewEmailPlaceholder"></list>	89 i18n-values="placeholder:addNewEmailPlaceholder"></list>

99 </div>	90 </div>

100 </div>	91 </div>

101 </div>	92 </div>

102 </div>	93 </div>
Ilya Sherman 2011/09/15 03:47:08 Let's have Phone and Email be side-by-side, since Let's have Phone and Email be side-by-side, since there's enough space for them now. James Hawkins 2011/09/16 03:23:28 I'd like to leave that for a followup CL. This is Show quoted text On 2011/09/15 03:47:08, Ilya Sherman wrote: > Let's have Phone and Email be side-by-side, since there's enough space for them > now. I'd like to leave that for a followup CL. This is large enough as is. Ilya Sherman 2011/09/16 03:57:09 Ok, but in that case please add a TODO or file a b Show quoted text On 2011/09/16 03:23:28, James Hawkins wrote: > On 2011/09/15 03:47:08, Ilya Sherman wrote: > > Let's have Phone and Email be side-by-side, since there's enough space for > them > > now. > > I'd like to leave that for a followup CL. This is large enough as is. Ok, but in that case please add a TODO or file a bug for this. James Hawkins 2011/09/16 04:23:21 Done. Show quoted text On 2011/09/16 03:57:09, Ilya Sherman wrote: > On 2011/09/16 03:23:28, James Hawkins wrote: > > On 2011/09/15 03:47:08, Ilya Sherman wrote: > > > Let's have Phone and Email be side-by-side, since there's enough space for > > them > > > now. > > > > I'd like to leave that for a followup CL. This is large enough as is. > > Ok, but in that case please add a TODO or file a bug for this. Done.
103 <div class="action-area button-strip">	94 <div class="action-area button-strip">

104 <button id="autofill-edit-address-cancel-button" type="reset"	95 <button id="autofill-edit-address-cancel-button" type="reset"

105 i18n-content="cancel"></button>	96 i18n-content="cancel"></button>

106 <button id="autofill-edit-address-apply-button" type="submit"	97 <button id="autofill-edit-address-apply-button" type="submit"

107 i18n-content="ok" disabled></button>	98 i18n-content="ok" disabled></button>

108 </div>	99 </div>

109 </div>	100 </div>

OLD	NEW