Support `browser` for Firefox extensions
danielrozenberg opened this issue · comments
Firefox extensions use browser, not chrome (though it supports both, MDN encourages using browser)
I tried just renaming all instances of chrome in the the files to browser but that didn't work - which is too bad because I wanted to send you a PR instead of filing an issue :)
Activating extension 'VaibhavVishal.web-extension' failed: Cannot find module '.browser/browserCompletionItemProvider'
Require stack:
- /home/daniel/.vscode/extensions/vaibhavvishal.web-extension-0.1.0/out/completionItemProviders.js
- /home/daniel/.vscode/extensions/vaibhavvishal.web-extension-0.1.0/out/extension.js
- /usr/share/code/resources/app/out/vs/loader.js
- /usr/share/code/resources/app/out/bootstrap-amd.js
- /usr/share/code/resources/app/out/bootstrap-fork.js
(yes I also renamed every file containing the name chrome to browser)
Update: it worked when I cloned your repo and did the same change. It just didn't work when I changed it in the already-compiled extension 🙃
@danielrozenberg does both chrome & firefox extensions supports browser?
I haven't looked at the project for a while, I will take a look on weekend
Chrome only supports chrome, Firefox supports browser and chrome (the latter is an alias to the former), other browsers (Edge, Safari) support browser, I don't know if they support chrome too...
hmm, in that case I will try to add autocomplete for both chrome & browser