Issue 6432 - Hide remove button for additional filter lists

README.md

 Shared Adblock Plus UI code
 ===========================
 
 The user interface elements defined in this repository will be used by various
 Adblock Plus products like Adblock Plus for Firefox. Their functionality can be
 tested within this repository, even though they might not work exactly the same
 as they will do in the final product.
 
 Installing dependencies
 -----------------------
@@ skipping 65 matching lines @@
 
 Remember, both `eslint` and `stylelint` can help fixing issues via `--fix` flag.
 
 You can try as example via [npx](https://medium.com/@maybekatz/introducing-npx-an-npm-package-runner-55f7d4bd282b)
 which should be provided automatically when you install `npm`.
 
 ```sh
 npx stylelint --fix skin/real-file-name.css
 ```
 
-Bundling
---------
+Bundling JS
+-----------
 
 As it is for the `desktop-options.js` case, bundling is done via `package.json`
 script entries.
 
-A dedicated script entry, such `bundle:desktop-options`,
-should perform the following operations:
-
-  * ensure source code passes linting
-  * ensure the bundle won't affect future linting operations
-  * ensure `browserify` uses `--node` and `--no-bundle-external` flags
-  * point at the entry point, and output in the top level folder
-
-Accordingly, this is what happens with the `bundle:desktop-options`:
-
-```sh
-# the && operator ensure each step is executed only
-# if the previous one didn't produce an error
-eslint ./js/**/*.js &&
-# browserify doesn't offer a way to prefix with text
-# the file is hence created with eslint disabled and a warning
-echo '/* eslint-disable */// BUNDLED FILE' > ./desktop-options.js &&
-# browserify take an entry point and bundle all its required files
-# outputting the result into ./desktop-options.js
-browserify --node --no-bundle-external js/desktop-options.js >> ./desktop-options.js
-```
-
-For a new bundle, i.e. `mobile-options.js`, simply use the same procedure
-but swap the `desktop-options` file/script name with `mobile-options`.
-
-The main `bundle` script should include each sub-bundle operation.
+A dedicated script entry, such `bundle:desktop-options.js`,
+should simply use the `bash:js` script, passing along
+the source file and the target.
+
+```js
+{
+  // example of a new bundle for the ./js/source.js file
+  "bundle:target.js": "npm run bash:js ./js/source.js ./target.js"
+}
+```
+
+The main `bundle` script should include each sub-bundle operation too.
+
+Bundling CSS
+------------
+
+As it is for the `desktop-options.css` case, bundling is done via `package.json`
+script entries.
+
+A dedicated script entry, such `bundle:desktop-options.css`,
+should simply use the `bash:css` script, passing along
+the source file and the target.
+
+```js
+{
+  // example of a new bundle for the ./css/source.scss file
+  "bundle:target.css": "npm run bash:css ./css/source.scss ./skin/target.css"
+}
+```
+
+In case there are dependencies, please ensure these are
+imported via `@import "dep.scss"` and not via `url(...)` syntax.
+
+As it is for JS bundles, the main `bundle` script should include each
+CSS bundle too.
 
 Watching
 --------
 
 While developing, it is convenient to bundle automatically
 each time a source file is changed.
 
 As a team, we agreed on restructuring JS code inside the js folder,
 so that is watched, and each bundled created, every time there is a changes.
 
@@ skipping 54 matching lines @@
 This is the implementation of the Adblock Plus options page which is
 the primary UI for changing settings and for managing filter lists.
 
 To aid testing, the behavior of this page is affected by a number of URL
 parameters:
 
 * `addonVersion`: sets addon version application parameter that is used for
   creating the link to the version-specific release notes
 * `addSubscription=true`: this parameter should trigger a dialog for adding
   subscriptions as initiated by clicking on an "abp:subscribe" link
-* `additionalSubscriptions=true`: Simulates scenario of persistent filter lists
-  that are preinstalled by administrators.
+* `additionalSubscriptions`: A comma-separated list of subscription URLs that
+  simulates scenario of persistent filter lists preinstalled by administrators.
 * `filterError=true`: causes filter validation to fail, showing validation
   errors when adding new filters on the options page
 * `blockedURLs`: a comma-separated list of URLs that should be considered
   blocked (necessary to test the check for blocked scripts in sharing buttons).
 * `downloadStatus`: sets downloadStatus parameter for filter lists, can be used
   to trigger various filter list download errors
 * `platform=chromium`: shows the opt-out for the developer tools panel
 * `showNotificationUI=true`: simulates user having opted-out of notifications
 
 
 [crowdin]: https://crowdin.com