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

Unified Diff: js/desktop-options.js

Issue 29712664: Issue 6432 - Hide remove button for additional filter lists (Closed)
Patch Set: Addressed Jeen's suggestion in the ticket Created March 2, 2018, 4:31 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: js/desktop-options.js
===================================================================
--- a/js/desktop-options.js
+++ b/js/desktop-options.js
@@ -237,8 +237,15 @@
{
control.setAttribute("aria-checked", item.disabled == false);
if (isAcceptableAds(item.url) && this == collections.filterLists)
+ {
control.disabled = true;
+ }
}
+ getPref("additional_subscriptions", (additionalSubscriptions) =>
+ {
+ if (additionalSubscriptions.includes(item.url))
+ element.classList.add("preconfigured");
+ });
let lastUpdateElement = element.querySelector(".last-update");
if (lastUpdateElement)

Powered by Google App Engine
This is Rietveld