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

Unified Diff: new-options.js

Issue 29338983: issue 3741 - Add "remove" option to list items in new options page (Closed)
Patch Set: Rebase to changeset #83 Created May 10, 2016, 2: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
Index: new-options.js
===================================================================
--- a/new-options.js
+++ b/new-options.js
@@ -333,7 +333,7 @@
{
id: "blocking-languages-table",
emptyText: "options_dialog_language_added_empty",
- onClick: toggleRemoveSubscription
+ onClick: toggleDisableSubscription
},
{
id: "blocking-languages-dialog-table",
@@ -359,7 +359,7 @@
[
{
id: "custom-list-table",
- onClick: toggleRemoveSubscription
+ onClick: toggleDisableSubscription
}
]);
collections.whitelist = new Collection(
@@ -476,7 +476,7 @@
var subscription = Object.create(null);
subscription.originalTitle = element.getAttribute("title");
subscription.url = element.getAttribute("url");
- subscription.disabled = null;
+ subscription.disabled = true;
subscription.downloadStatus = null;
subscription.homepage = null;
var recommendation = Object.create(null);

Powered by Google App Engine
This is Rietveld