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

Unified Diff: README.md

Issue 29559679: Issue 5803 - rename new-options to options (Closed)
Patch Set: Created Sept. 29, 2017, 4:34 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') | ext/background.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -16,7 +16,7 @@
to messages sent by UI code.
* `background.html`, `background.js`: Test implementation of the background
page, should *not be imported*.
- * `new-options.html`, `new-options.js`: Options page, see below
+ * `options.html`, `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
@@ -83,10 +83,10 @@
subscriptions as initiated by clicking on an "abp:subscribe" link
* `showPageOptions=true`: shows page-specific options
-new-options.html
+options.html
------------
-This is the new implementation of the Adblock Plus options page which will be
+This is the implementation of the Adblock Plus options page which will be
Thomas Greiner 2017/09/29 16:51:09 Detail: Since we're removing "new" from this sente
saroyanm 2017/09/29 17:09:32 Agree, Done.
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
« no previous file with comments | « no previous file | ext/background.js » ('j') | ext/background.js » ('J')

Powered by Google App Engine
This is Rietveld