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

Unified Diff: README.md

Issue 29340737: Issue 3968 - Add "new-" prefix to option file names (Closed)
Patch Set: Updated sentence about firstRun parameters for consistency. Created April 22, 2016, 12:23 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 | « no previous file | ext/background.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
diff --git a/README.md b/README.md
index 0e81b7fcb5a33ed2ffa072579b7a2014610de670..a01d6588bf672e08fce7b974cebe0d32f4db809b 100644
--- a/README.md
+++ b/README.md
@@ -16,7 +16,7 @@ Directory structure
to messages sent by UI code.
* `background.html`, `background.js`: Test implementation of the background
page, should *not be imported*.
- * `options.html`, `options.js`: Options page, see below
+ * `new-options.html`, `new-options.js`: Options page, see below
* `subscriptions.xml`: Test subscription data, should *not be imported*
* `locale` directory: Localized strings, with one directory per locale. The
Firefox format for locale identifiers is used (xx-YY where xx is the language
@@ -60,7 +60,8 @@ whenever changes are applied automatically to user's Adblock Plus configuration.
This will usually happen when the user first installs Adblock Plus (initial
setup), but it can also happen in case the user's settings get lost.
-The behavior of this page is affected by a number of URL parameters:
+To aid testing, the behavior of this page is affected by a number of URL
+parameters:
* `platform`, `platformVersion`, `application`, `applicationVersion`: sets
application parameters that are normally determined by Adblock Plus. Using
@@ -70,13 +71,14 @@ The behavior of this page is affected by a number of URL parameters:
* `blockedURLs`: a comma-separated list of URLs that should be considered
blocked (necessary to test the check for blocked scripts in sharing buttons).
-options.html
+new-options.html
------------
-This is the implementation of the Adblock Plus options page which is the primary
-UI for changing settings and for managing filter lists.
+This is the new implementation of the Adblock Plus options page which will be
+the primary UI for changing settings and for managing filter lists.
-The behavior of this page is affected by a number of URL parameters:
+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
« no previous file with comments | « no previous file | ext/background.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld