Delta Between Two Patch Sets: new-options.html
Issue 29411555:
Issue 5169 - Add whitelisted tab to the new options page (Closed)
Left Patch Set: Addressed latest comments
Created May 22, 2017, 9:35 a.m.
Right Patch Set: Fixed the TYPO
Created July 3, 2017, 4:10 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« locale/en-US/new-options.json ('k')
|
new-options.js » ('j')
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <!-- | 2 <!-- |
| 3 - This file is part of Adblock Plus <https://adblockplus.org/>, | 3 - This file is part of Adblock Plus <https://adblockplus.org/>, |
| 4 - Copyright (C) 2006-2017 eyeo GmbH | 4 - Copyright (C) 2006-2017 eyeo GmbH |
| 5 - | 5 - |
| 6 - Adblock Plus is free software: you can redistribute it and/or modify | 6 - Adblock Plus is free software: you can redistribute it and/or modify |
| 7 - it under the terms of the GNU General Public License version 3 as | 7 - it under the terms of the GNU General Public License version 3 as |
| 8 - published by the Free Software Foundation. | 8 - published by the Free Software Foundation. |
| 9 - | 9 - |
| 10 - Adblock Plus is distributed in the hope that it will be useful, | 10 - Adblock Plus is distributed in the hope that it will be useful, |
| (...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 166 </ul> | 166 </ul> |
| 167 </div> | 167 </div> |
| 168 </div> | 168 </div> |
| 169 </div> | 169 </div> |
| 170 </div> | 170 </div> |
| 171 | 171 |
| 172 <!-- Whitelist tab content --> | 172 <!-- Whitelist tab content --> |
| 173 <div id="content-whitelist" role="tabpanel" aria-labelledby="tab-whiteli st"> | 173 <div id="content-whitelist" role="tabpanel" aria-labelledby="tab-whiteli st"> |
| 174 <h1 class="i18n_options_tab_whitelist"></h1> | 174 <h1 class="i18n_options_tab_whitelist"></h1> |
| 175 <p class="i18n_options_whitelist_description"></p> | 175 <p class="i18n_options_whitelist_description"></p> |
| 176 <form action=""> | 176 <form> |
|
Thomas Greiner
2017/05/26 11:10:39
Detail: Do we need the "action" attribute? We'll n
saroyanm
2017/05/31 08:30:26
Done.
saroyanm
2017/05/31 08:30:26
Interesting MDN still uses empty action attribute
| |
| 177 <input id="whitelisting-textbox" type="text"> | 177 <input id="whitelisting-textbox" type="text"> |
| 178 <button id="whitelisting-add-button" | 178 <button id="whitelisting-add-button" |
| 179 type="submit" | 179 type="submit" |
| 180 data-action="add-domain-exception" | 180 data-action="add-domain-exception" |
| 181 class="i18n_options_whitelist_add" disabled> | 181 class="i18n_options_whitelist_add" disabled> |
| 182 </button> | 182 </button> |
| 183 <div id="whitelisting-validation"></div> | 183 <div id="whitelisting-validation"></div> |
| 184 </form> | 184 </form> |
| 185 <ul id="whitelisting-table" class="table list"> | 185 <ul id="whitelisting-table" class="table list"> |
| 186 <template> | 186 <template> |
| (...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 413 <div class="url"></div> | 413 <div class="url"></div> |
| 414 <button class="i18n_options_dialog_predefined_confirm default-focus" data-action="add-predefined-subscription"></button> | 414 <button class="i18n_options_dialog_predefined_confirm default-focus" data-action="add-predefined-subscription"></button> |
| 415 </div> | 415 </div> |
| 416 </div> | 416 </div> |
| 417 </div> | 417 </div> |
| 418 <!-- Placeholder element to determine when to wrap focus around --> | 418 <!-- Placeholder element to determine when to wrap focus around --> |
| 419 <span class="focus-last" tabindex="0"></span> | 419 <span class="focus-last" tabindex="0"></span> |
| 420 </div> | 420 </div> |
| 421 </body> | 421 </body> |
| 422 </html> | 422 </html> |
| LEFT | RIGHT |
« locale/en-US/new-options.json ('k')
|
new-options.js » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
