Dan Callahan [:callahad] Reporter
	Description • 9 years ago

This metabug depends on all of the bugs required to make the Reddit Enhancement Suite work as a WebExtension without modification.

Resources:

- Homepage: http://redditenhancementsuite.com/

- GitHub Repo: https://github.com/honestbleeps/Reddit-Enhancement-Suite/

- AMO Listing: https://addons.mozilla.org/en-US/firefox/addon/reddit-enhancement-suite/

- Chrome Listing: https://chrome.google.com/webstore/detail/reddit-enhancement-suite/kbmfpngjjgdllneeigpgjifpgocmfgmb 

- Proof-of-concept port / kludge: https://github.com/callahad/RES-WebExtension

	Steve Sobel
	Comment 1 • 8 years ago

Hey, RES author here! It's awesome to see this in bugzilla - and I really appreciate that you're using my extension as an example for making sure WebExtensions is done right. It's an honor!

A quick note on the dependencies for options screens:

Reddit Enhancement Suite uses its own options/settings screen that's displayed via HTML/CSS/JS right on reddit (it uses simple storage in Firefox, chrome's localStorage in Chrome, etc as a datastore for those options) because the complexity of the options requires more complex UX than any of the native browser APIs provide.

For that reason, although it's likely important to Firefox WebExtensions as a whole, I believe that there is no dependency for either #1250784 or #1212685 for Reddit Enhancement Suite to work properly as a WebExtension. RES doesn't use either of those.

Thanks!

	Dan Callahan [:callahad] Reporter
	Comment 2 • 8 years ago

Removing Bug 1208334 as a blocker; we've implemented the history.* APIs that RES needed.

Adding Bug 1213473 as a blocker; RES now needs chrome.runtime.lasterror.

	Dan Callahan [:callahad] Reporter
	Comment 3 • 8 years ago

Removing Bug 1213473 as a blocker; according to https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/runtime/lastError we *do* support runtime.lastError, and have since 47. Oops. :)

	Dan Callahan [:callahad] Reporter
	Comment 4 • 8 years ago

Removing Bug 1213473 as a blocker; according to https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/runtime/lastError we *do* support runtime.lastError, and have since 47. Oops. :)

	Andy McKay
	Comment 5 • 8 years ago

Apart from permissions, is there anything else here? Could we close this?

	Dan Callahan [:callahad] Reporter
	Comment 6 • 8 years ago

I think we're set, need to verify.

	Andy McKay
	Comment 7 • 8 years ago

Assuming closed then, if that's not correct callahad, please re-open.

	Andrei Petcu
	Comment 8 • 8 years ago

AMO seems not to provide this as a WebExtension. When will this be available to the public?

	Andy McKay
	Comment 9 • 8 years ago

That's up to the add-on developer to upload.