Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: options.html

Issue 29323156: Issue 2410 - Improved accessibility of dialogs in options page (Closed)
Left Patch Set: Rebased to f8e3e591fcbb Created Sept. 24, 2015, 10:37 a.m.
Right Patch Set: Created Oct. 2, 2015, 2:11 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | options.js » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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-2015 Eyeo GmbH 4 - Copyright (C) 2006-2015 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 305 matching lines...) Expand 10 before | Expand all | Expand 10 after
316 <h3 class="i18n_options_dialog_language_added"></h3> 316 <h3 class="i18n_options_dialog_language_added"></h3>
317 <ul id="blocking-languages-dialog-table" class="table list"> 317 <ul id="blocking-languages-dialog-table" class="table list">
318 <template> 318 <template>
319 <span class="display"></span> 319 <span class="display"></span>
320 </template> 320 </template>
321 </ul> 321 </ul>
322 </div> 322 </div>
323 <div id="other-language" class="dialog-content-block"> 323 <div id="other-language" class="dialog-content-block">
324 <div> 324 <div>
325 <label for="find-language" class="i18n_options_dialog_language_oth er"></label> 325 <label for="find-language" class="i18n_options_dialog_language_oth er"></label>
326 <input type="search" id="find-language" class="default-action" /> 326 <input type="search" id="find-language" class="default-focus" />
saroyanm 2015/10/01 16:31:40 As far as I can see default-action is used for def
Thomas Greiner 2015/10/02 10:27:28 Done.
327 </div> 327 </div>
328 <ul id="all-lang-table" class="table list"> 328 <ul id="all-lang-table" class="table list">
329 <template> 329 <template>
330 <button class="button-add control"> 330 <button class="button-add control">
331 +<span></span> 331 +<span></span>
332 </button> 332 </button>
333 <span class="display"></span> 333 <span class="display"></span>
334 </template> 334 </template>
335 </ul> 335 </ul>
336 </div> 336 </div>
337 </div> 337 </div>
338 <!-- Add custom subscription --> 338 <!-- Add custom subscription -->
339 <div id="dialog-content-custom" class="dialog-content"> 339 <div id="dialog-content-custom" class="dialog-content">
340 <div class="dialog-content-block"> 340 <div class="dialog-content-block">
341 <div> 341 <div>
342 <label for="blockingList-textbox" class="i18n_options_dialog_custo m_subscription_title"></label> 342 <label for="blockingList-textbox" class="i18n_options_dialog_custo m_subscription_title"></label>
343 <input id="blockingList-textbox" type="text" class="default-action " placeholder="www.example.com/blockinglist.txt" /> 343 <input id="blockingList-textbox" type="text" class="default-focus" placeholder="www.example.com/blockinglist.txt" />
344 </div> 344 </div>
345 <button class="i18n_options_dialog_custom_import" data-action="impor t-subscription"></button> 345 <button class="i18n_options_dialog_custom_import" data-action="impor t-subscription"></button>
346 </div> 346 </div>
347 <div class="dialog-content-block"> 347 <div class="dialog-content-block">
348 <h3 class="i18n_options_dialog_edit_own_list"></h3> 348 <h3 class="i18n_options_dialog_edit_own_list"></h3>
349 <button class="i18n_options_dialog_create_own_list" data-action="clo se-dialog,switch-tab,edit-custom-filters" data-tab="advanced"></button> 349 <button class="i18n_options_dialog_create_own_list" data-action="clo se-dialog,switch-tab,edit-custom-filters" data-tab="advanced"></button>
350 </div> 350 </div>
351 </div> 351 </div>
352 <!-- Add predefined subscription --> 352 <!-- Add predefined subscription -->
353 <div id="dialog-content-predefined" class="dialog-content"> 353 <div id="dialog-content-predefined" class="dialog-content">
354 <div class="dialog-content-block"> 354 <div class="dialog-content-block">
355 <h3></h3> 355 <h3></h3>
356 <div class="url"></div> 356 <div class="url"></div>
357 <button class="i18n_options_dialog_predefined_confirm default-action " data-action="add-predefined-subscription"></button> 357 <button class="i18n_options_dialog_predefined_confirm default-focus" data-action="add-predefined-subscription"></button>
358 </div> 358 </div>
359 </div> 359 </div>
360 </div> 360 </div>
361 <!-- Placeholder element to determine when to wrap focus around --> 361 <!-- Placeholder element to determine when to wrap focus around -->
362 <span class="focus-last" tabindex="0"></span> 362 <span class="focus-last" tabindex="0"></span>
363 </div> 363 </div>
364 </body> 364 </body>
365 </html> 365 </html>
LEFTRIGHT
« no previous file | options.js » ('j') | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld