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

Unified Diff: new-options.html

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Patch Set: Rebaseed to changeset 84 Created June 10, 2016, 4:36 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « locale/en-US/new-options.json ('k') | new-options.js » ('j') | new-options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: new-options.html
===================================================================
--- a/new-options.html
+++ b/new-options.html
@@ -89,8 +89,9 @@
</div>
<ul id="blocking-languages-table" class="table list">
<template>
- <button role="checkbox" class="control"></button>
+ <button data-action="toggle-disable-subscription" role="checkbox" class="control" title="toggle"></button>
Thomas Greiner 2016/06/16 10:42:22 The titles need to be translatable as well.
saroyanm 2016/06/16 18:22:59 Done.
<label class="display"></label>
+ <button data-action="remove-subscription" class="delete control" title="remove"></button>
</template>
</ul>
<div class="controls">
@@ -111,19 +112,19 @@
<div id="custom-wrapper">
<ul id="recommend-list-table" class="table list">
<template>
- <button role="checkbox" class="control"></button>
+ <button data-action="toggle-remove-subscription" role="checkbox" class="control"></button>
<div class="fill">
<label class="display"
data-tooltip="options_more_%value%_tooltip"
data-tooltip-native></label>
</div>
- <span class="i18n_options_popular popular"></span>
</template>
</ul>
<ul id="custom-list-table" class="table list">
<template>
- <button role="checkbox" class="control"></button>
+ <button data-action="toggle-disable-subscription" role="checkbox" class="control" title="toggle"></button>
<label class="display"></label>
+ <button data-action="remove-subscription" class="delete control" title="remove"></button>
</template>
</ul>
</div>
@@ -148,7 +149,7 @@
</div>
<ul id="acceptableads-table" class="table list">
<template>
- <button role="checkbox" class="control"></button>
+ <button data-action="toggle-remove-subscription" role="checkbox" class="control"></button>
<label class="display"></label>
</template>
</ul>
@@ -163,8 +164,8 @@
</div>
<ul id="whitelisting-table" class="table list">
<template>
- <button class="delete control"></button>
<label class="display"></label>
+ <button data-action="remove-filter" class="delete control" title="remove"></button>
</template>
</ul>
<div class="controls">
@@ -241,7 +242,7 @@
</div>
<ul class="table cols" id="all-filter-lists-table">
<template>
- <button role="checkbox" class="control"></button>
+ <button data-action="toggle-disable-subscription" role="checkbox" class="control"></button>
<div>
<label data-action="open-context-menu" class="display"></label>
<div data-action="open-context-menu" class="arrow">
@@ -372,7 +373,7 @@
</div>
<ul id="all-lang-table" class="table list">
<template>
- <button class="button-add control">
+ <button data-action="add-language-subscription" class="button-add control">
+<span class="i18n_options_button_add"></span>
</button>
<label class="display"></label>
« no previous file with comments | « locale/en-US/new-options.json ('k') | new-options.js » ('j') | new-options.js » ('J')

Powered by Google App Engine
This is Rietveld