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

Side by Side Diff: README.md

Issue 29680702: Issue 6008 - Expose updates version via lib/prefs.js (Closed)
Patch Set: Renamed lib/ui.js to lib/prefs.js Created Jan. 30, 2018, 6:18 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | lib/prefs.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 Shared Adblock Plus UI code 1 Shared Adblock Plus UI code
2 =========================== 2 ===========================
3 3
4 The user interface elements defined in this repository will be used by various 4 The user interface elements defined in this repository will be used by various
5 Adblock Plus products like Adblock Plus for Firefox. Their functionality can be 5 Adblock Plus products like Adblock Plus for Firefox. Their functionality can be
6 tested within this repository, even though they might not work exactly the same 6 tested within this repository, even though they might not work exactly the same
7 as they will do in the final product. 7 as they will do in the final product.
8 8
9 Directory structure 9 Directory structure
10 ------------------- 10 -------------------
11 11
12 * Top-level files: 12 * Top-level files:
13 * `firstRun.html` and `firstRun.js`: First-run page, see below 13 * `firstRun.html` and `firstRun.js`: First-run page, see below
14 * `i18n.js`: Localization functions, should be included by all pages. 14 * `i18n.js`: Localization functions, should be included by all pages.
15 * `messageResponder.js`: Script to be used on the background page to respond 15 * `messageResponder.js`: Script to be used on the background page to respond
16 to messages sent by UI code. 16 to messages sent by UI code.
17 * `background.html`, `background.js`: Test implementation of the background 17 * `background.html`, `background.js`: Test implementation of the background
18 page, should *not be imported*. 18 page, should *not be imported*.
19 * `desktop-options.html`, `desktop-options.js`: Options page, see below 19 * `desktop-options.html`, `desktop-options.js`: Options page, see below
20 * `subscriptions.xml`: Test subscription data, should *not be imported* 20 * `subscriptions.xml`: Test subscription data, should *not be imported*
21 * `polyfill.js`: Browser API polyfills, should *not be imported* 21 * `polyfill.js`: Browser API polyfills, should *not be imported*
22 * `lib` directory: Modules to be used on the background page to expose
23 UI-related functionality.
22 * `locale` directory: Localized strings, with one directory per locale. The 24 * `locale` directory: Localized strings, with one directory per locale. The
23 Firefox format for locale identifiers is used (xx-YY where xx is the language 25 Firefox format for locale identifiers is used (xx-YY where xx is the language
24 code and YY the optional region code). The localization strings themselves are 26 code and YY the optional region code). The localization strings themselves are
25 stored in the JSON format, like the one used by Chrome extensions. There is 27 stored in the JSON format, like the one used by Chrome extensions. There is
26 one JSON file per HTML page, file names of HTML page and JSON file should 28 one JSON file per HTML page, file names of HTML page and JSON file should
27 match. 29 match.
28 * `skin` directory: CSS files and any additional resources (images and fonts) 30 * `skin` directory: CSS files and any additional resources (images and fonts)
29 required for these. 31 required for these.
30 * `ext` directory: Test implementation of the abstraction layer. This one should 32 * `ext` directory: Test implementation of the abstraction layer. This one should
31 *not to be imported*, these files should rather be replaced by 33 *not to be imported*, these files should rather be replaced by
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 Linting 114 Linting
113 ------- 115 -------
114 116
115 You can lint the code using [ESLint](http://eslint.org). 117 You can lint the code using [ESLint](http://eslint.org).
116 118
117 eslint *.js lib ext 119 eslint *.js lib ext
118 120
119 You will need to set up ESLint and our configuration first, see 121 You will need to set up ESLint and our configuration first, see
120 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo) 122 [eslint-config-eyeo](https://hg.adblockplus.org/codingtools/file/tip/eslint-conf ig-eyeo)
121 for more information. 123 for more information.
OLDNEW
« no previous file with comments | « no previous file | lib/prefs.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld