Issue 6432 - Hide remove button for additional filter lists

css/desktop-options.scss

Index: css/desktop-options.scss
===================================================================
--- a/css/desktop-options.scss
+++ b/css/desktop-options.scss
@@ -115,6 +115,12 @@
   cursor: pointer;
 }
 
+button:disabled,
+button[aria-disabled="true"]
+{
+  cursor: default;
+}
+
 /* Ignore .icon to avoid overriding "specific" (primary, secondary) styles */
 button.primary:not(.icon),
 .button.primary:not(.icon)
@@ -268,7 +274,7 @@
   border: 0rem;
   margin: 0.2rem;
   border-radius: 2px;
-  background-color: #ccc;
+  background-color: #b3b3b3;

[Thomas Greiner, 2018/03/08 19:28:19]

Detail: Isn't this redundant since we're setting `fill="#b3b3b3"` in the SVG
anyway?

[saroyanm, 2018/03/09 11:23:28]

It is not because we do not have dedicated disable state for the Accceptable Ads
button, this is something make sense to introduce after we update the specs.

This implementation is a hack in order to not introduce a new button, this is
something that needs to be change as part of Jeens change to the specs.

I'll change this color back to #ccc not to block this review with the color
discussion.

[Thomas Greiner, 2018/03/09 17:24:22]

Acknowledged.

 }
 
 button[role="checkbox"].icon:not(.toggle)::before
@@ -282,6 +288,11 @@
   background-image: url(icons/checkbox.svg?on#on);
 }
 
+button[role="checkbox"][aria-checked="true"].icon:disabled:not(.toggle)::before
+{
+  background-image: url(icons/checkbox.svg?on-disabled#on-disabled);
+}
+
 button[role="checkbox"].icon.toggle::before
 {
   background-image: url(icons/toggle.svg?on#on);
@@ -1073,6 +1084,11 @@
   font-weight: 400;
 }
 
+li.preconfigured [data-hide="preconfigured"]
+{
+  display: none !important;
+}
+
 /*
   Tooltips
 */