Support WebExtensions of Firefox extension

[mitsugu]

Firefox 57 will be released shortly. Firefox 57 will only work with WebExtensions extensions.
So, please release the WebExtensions version of LiveReload.

[erikw]

Yup, FX57 out now :(

[twolfson]

I'm looking into this now. It seems to be rather straightforward; we can copy/paste Chrome's setup and reuse it in Firefox since Chrome extension ~= Browser extension now. I have a proof of concept working by replacing all of the chrome globals with browser but it seems to not persist the reload after the first time

[twolfson]

I've got it working and opened a PR (#49) to fix up the changes. Since this repo seems relatively inactive, I also posted the forked version to addons.mozilla.org. They have an auto-checker which processed it within 7 minutes ✨

https://addons.mozilla.org/en-US/firefox/addon/livereload-web-extension/

[erikw]

@twolfson h@cKor!!

I tried it out and indeed it works!! Thanks for fixing my daily workflow! :) I use this to observe unit test reports as the are continuously being run in the background as I develop :).

I only noticed one problem, that the icon does not change to show that LiveReload on the current page is active after clicking on it. I recall from pre-Firefox 57, the icon would become a bit red in the middle or something.

Edit.
I see now that the little circle in the middle becomes filled after activating it, but it's waay to hard to notice until looking closely at pixel-level :P

[twolfson]

Yea, it's not the best UX but it is what Chrome and Safari are using so I stuck with it =/

• Chrome: https://github.com/livereload/livereload-extensions/blob/0a8edb9cc7008249c23718386e19d99230e81179/Chrome/LiveReload/IconDisabled%402x.png
• Firefox: https://github.com/livereload/livereload-extensions/blob/0a8edb9cc7008249c23718386e19d99230e81179/Firefox/skin/IconEnabled.png
• Safari: https://github.com/livereload/livereload-extensions/blob/0a8edb9cc7008249c23718386e19d99230e81179/LiveReload.safariextension/IconEnabled%402x.png

[argordmel]

Hello

I've a problem with the https protocol in my localhost. The extension don't detect the current protocol, load always with http.

Thanks.

[twolfson]

@argordmel Could you file an issue in the fork? I don't want to bog down this issue with one-off bugs =/

https://github.com/twolfson/livereload-extensions

[lorenzos]

@twolfson Thank you for the fork and the AMO release. Anyway, the state of the icon is really unnoticeable! Please, consider adding some color in the active state!

[erikw]

I created an issue about the unnoticeable icon at the forked Github repo!

twolfson#1