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

Unified Diff: README.md

Issue 29458601: Issue 5315 - Add support for Microsoft Edge (Closed)
Patch Set: Rebase the README.md, compress images, adress the comments Created June 14, 2017, 3:33 a.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/common.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.md
===================================================================
--- a/README.md
+++ b/README.md
@@ -1,10 +1,10 @@
-Adblock Plus for Chrome, Opera and Firefox
+Adblock Plus for Chrome, Opera, Microsoft Edge and Firefox
==========================================
Sebastian Noack 2017/06/14 05:13:39 Nit: Please use as many equal signs here as the li
This repository contains the platform-specific Adblock Plus source code for
-Chrome, Opera and Firefox. It can be used to build Adblock Plus for these
-platforms, generic Adblock Plus code will be extracted from other repositories
-automatically (see _dependencies_ file).
+Chrome, Opera, Microsoft Edge and Firefox. It can be used to build
+Adblock Plus for these platforms, generic Adblock Plus code will be extracted
+from other repositories automatically (see _dependencies_ file).
Note that the Firefox extension built from this repository is the new
[WebExtension](https://developer.mozilla.org/en-US/Add-ons/WebExtensions).
@@ -29,13 +29,19 @@
./build.py -t chrome build -k adblockpluschrome.pem
./build.py -t gecko-webext build
+ ./build.py -t edge build
Sebastian Noack 2017/06/14 05:13:39 Nit: In the human-readable description we refer to
This will create a build with a name in the form
-_adblockpluschrome-1.2.3.nnnn.crx_ or _adblockplusfirefox-1.2.3.nnnn.xpi_
+_adblockpluschrome-1.2.3.nnnn.crx_ or _adblockplusfirefox-1.2.3.nnnn.xpi_ or
Sebastian Noack 2017/06/14 05:13:39 Nit: Don't use multiple "or", use "a, b or c" inst
+_adblockplusedge-1.2.3.nnnn.appx_.
Note that you don't need an existing signing key for Chrome, a new key
will be created automatically if the file doesn't exist.
+The Microsoft Edge build _adblockplusedge-1.2.3.nnnn.appx_ is unsigned and
+is only useful for uploading into Windows Store, where it will be signed. For
+testing use the devenv build.
+
The Firefox extension will be unsigned, and therefore is mostly only useful for
upload to Mozilla Add-ons. You can also also load it for testing purposes under
_about:debugging_ or by disabling signature enforcement in Firefox Nightly.
@@ -47,12 +53,18 @@
./build.py -t chrome devenv
./build.py -t gecko-webext devenv
+ ./build.py -t edge devenv
This will create a _devenv.*_ directory in the repository. You can load the
directory as an unpacked extension, under _chrome://extensions_ in Chrome,
-or under _about:debugging_ in Firefox. After making changes to the
-source code re-run the command to update the development environment, the
-extension should reload automatically after a few seconds.
+under _about:debugging_ in Firefox or in Extensions menu in Edge, after enabling
Sebastian Noack 2017/06/14 05:13:38 "... in the _Extensions_ menu in Microsoft Edge ..
+extension development features in _about:flags_. After making changes to the
+source code re-run the command to update the development environment. In Chrome
+and Firefox the extension should reload automatically after a few seconds.
+
+Edge build does not automatically detect changes, so after rebuilding the
Sebastian Noack 2017/06/14 05:13:39 "Builds for Microsoft Edge do not ..."
+extension you should manually force reloading it in Edge by hitting the
+_Reload Extension_ button.
Running the unit tests
----------------------
« no previous file with comments | « no previous file | ext/background.js » ('j') | ext/common.js » ('J')

Powered by Google App Engine
This is Rietveld