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

Unified Diff: options.html

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Patch Set: Created March 23, 2016, 4:17 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/options.json ('k') | options.js » ('j') | options.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: options.html
===================================================================
--- a/options.html
+++ b/options.html
@@ -99,6 +99,7 @@
<template>
<button role="checkbox" class="control"></button>
<span class="display"></span>
+ <button data-action="remove-subscription" class="delete control"></button>
</template>
</ul>
<div class="controls">
@@ -124,13 +125,13 @@
<template>
<button role="checkbox" class="control"></button>
<span class="display"></span>
- <span class="i18n_options_popular popular"></span>
</template>
</ul>
<ul id="custom-list-table" class="table list">
<template>
<button role="checkbox" class="control"></button>
<span class="display"></span>
+ <button data-action="remove-subscription" class="delete control"></button>
</template>
</ul>
</div>
@@ -183,8 +184,8 @@
</div>
<ul id="whitelisting-table" class="table list">
<template>
+ <span class="display"></span>
<button class="delete control"></button>
- <span class="display"></span>
</template>
</ul>
<div class="controls">
« no previous file with comments | « locale/en-US/options.json ('k') | options.js » ('j') | options.js » ('J')

Powered by Google App Engine
This is Rietveld