Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 <div class="page hidden" id="autoFillEditAddressOverlay"> | 1 <div class="page hidden" id="autoFillEditAddressOverlay"> |
| 2 <h1 id="autoFillAddressTitle"></h1> | 2 <h1 id="autoFillAddressTitle"></h1> |
| 3 <div class="content-area"> | 3 <div class="content-area"> |
| 4 <div class="input"> | 4 <div class="input"> |
| 5 <label id="fullNameLabel"> | 5 <label id="fullNameLabel"> |
| 6 <div><span i18n-content="fullNameLabel"></span></div> | 6 <div><span i18n-content="fullNameLabel"></span></div> |
| 7 <input type="text" class="form" class="form" id="fullName"> | 7 <input type="text" class="form" class="form" id="fullName"> |
| 8 </label> | 8 </label> |
| 9 </div> | 9 </div> |
| 10 <div class="input"> | 10 <div class="input"> |
|
dhollowa
2011/02/17 00:55:28
As discussed, country should follow other address
Ilya Sherman
2011/02/17 23:09:11
Done.
| |
| 11 <label id="country-label"> | |
| 12 <div><span i18n-content="country-label"></span></div> | |
| 13 <select id="country"></select> | |
| 14 </label> | |
| 15 </div> | |
| 16 <div class="input"> | |
| 11 <label id="companyNameLabel"> | 17 <label id="companyNameLabel"> |
|
dhollowa
2011/02/17 00:55:28
The new id and i18n-content values seem inconsiste
Ilya Sherman
2011/02/17 23:09:11
Hyphens, but James seemed to want to change the re
| |
| 12 <div><span i18n-content="companyNameLabel"></span></div> | 18 <div><span i18n-content="companyNameLabel"></span></div> |
| 13 <input type="text" class="form" class="form" id="companyName"> | 19 <input type="text" class="form" class="form" id="companyName"> |
| 14 </label> | 20 </label> |
| 15 </div> | 21 </div> |
| 16 <div class="input"> | 22 <div class="input"> |
| 17 <label id="addrLine1Label"> | 23 <label id="addrLine1Label"> |
| 18 <div><span i18n-content="addrLine1Label"></span></div> | 24 <div><span i18n-content="addrLine1Label"></span></div> |
| 19 <input type="text" class="form" class="form" id="addrLine1"> | 25 <input type="text" class="form" class="form" id="addrLine1"> |
| 20 </label> | 26 </label> |
| 21 </div> | 27 </div> |
| 22 <div class="input"> | 28 <div class="input"> |
| 23 <label id="addrLine2Label"> | 29 <label id="addrLine2Label"> |
| 24 <div><span i18n-content="addrLine2Label"></span></div> | 30 <div><span i18n-content="addrLine2Label"></span></div> |
| 25 <input type="text" class="form" class="form" id="addrLine2"> | 31 <input type="text" class="form" class="form" id="addrLine2"> |
| 26 </label> | 32 </label> |
| 27 </div> | 33 </div> |
| 28 <div class="table"> | 34 <div class="table"> |
| 29 <div class="row"> | 35 <div class="row"> |
| 30 <div class="input cell"> | 36 <div class="input cell"> |
| 31 <label id="cityLabel" for="city"> | 37 <label id="cityLabel" for="city"> |
| 32 <span i18n-content="cityLabel"></span> | 38 <span i18n-content="cityLabel"></span> |
| 33 </label> | 39 </label> |
| 34 </div> | 40 </div> |
| 35 <div class="input cell"> | 41 <div class="input cell"> |
| 36 <label id="stateLabel" for="state"> | 42 <label id="state-label" for="state"></label> |
|
arv (Not doing code reviews)
2011/02/21 03:09:27
You should probably fix the other cases too or thi
Ilya Sherman
2011/02/23 01:17:15
Split out to http://codereview.chromium.org/655200
| |
| 37 <span i18n-content="stateLabel"></span> | |
| 38 </label> | |
| 39 </div> | 43 </div> |
| 40 <div class="input cell"> | 44 <div class="input cell"> |
| 41 <label id="zipCodeLabel" for="zipCode"> | 45 <label id="postal-code-label" for="postal-code"></label> |
| 42 <span i18n-content="zipCodeLabel"></span> | |
| 43 </label> | |
| 44 </div> | 46 </div> |
| 45 </div> | 47 </div> |
| 46 <div class="row"> | 48 <div class="row"> |
| 47 <div class="input cell"> | 49 <div class="input cell"> |
| 48 <input type="text" class="form" id="city"> | 50 <input type="text" class="form" id="city"> |
| 49 </div> | 51 </div> |
| 50 <div class="input cell"> | 52 <div class="input cell"> |
| 51 <input type="text" class="form" id="state"> | 53 <input type="text" class="form" id="state"> |
| 52 </div> | 54 </div> |
| 53 <div class="input cell"> | 55 <div class="input cell"> |
| 54 <input type="text" class="form" id="zipCode"> | 56 <input type="text" class="form" id="postal-code"> |
| 55 </div> | 57 </div> |
| 56 </div> | 58 </div> |
| 57 </div> | 59 </div> |
| 58 <div class="input"> | |
| 59 <label id="countryLabel"> | |
| 60 <div><span i18n-content="countryLabel"></span></div> | |
| 61 <input type="text" class="form" class="form" id="country"> | |
| 62 </label> | |
| 63 </div> | |
| 64 <div class="table"> | 60 <div class="table"> |
| 65 <div class="row"> | 61 <div class="row"> |
| 66 <div class="input cell"> | 62 <div class="input cell"> |
| 67 <label id="phoneLabel" for="phone"> | 63 <label id="phoneLabel" for="phone"> |
| 68 <span i18n-content="phoneLabel"></span> | 64 <span i18n-content="phoneLabel"></span> |
| 69 </label> | 65 </label> |
| 70 </div> | 66 </div> |
| 71 <div class="input cell"> | 67 <div class="input cell"> |
| 72 <label id="faxLabel" for="fax"> | 68 <label id="faxLabel" for="fax"> |
| 73 <span i18n-content="faxLabel"></span> | 69 <span i18n-content="faxLabel"></span> |
| (...skipping 18 matching lines...) Expand all Loading... | |
| 92 </div> | 88 </div> |
| 93 <div class="action-area"> | 89 <div class="action-area"> |
| 94 <div class="button-strip"> | 90 <div class="button-strip"> |
| 95 <button type="reset" id="autoFillEditAddressCancelButton" | 91 <button type="reset" id="autoFillEditAddressCancelButton" |
| 96 i18n-content="cancel"></button> | 92 i18n-content="cancel"></button> |
| 97 <button type="submit" id="autoFillEditAddressApplyButton" disabled | 93 <button type="submit" id="autoFillEditAddressApplyButton" disabled |
| 98 i18n-content="ok"></button> | 94 i18n-content="ok"></button> |
| 99 </div> | 95 </div> |
| 100 </div> | 96 </div> |
| 101 </div> | 97 </div> |
| OLD | NEW |