-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Mozilla Firefox browser #58
Comments
hey, I'd like to pick this up if that's ok? |
Assigned! |
damn that was speedy! 🏎️ |
🏎️💨 |
Hi, what's the status? Can I help in any way? I'm open to fixing any issue in the project so please let me know. |
Hey @codingfarhan I picked this up and started looking - i think it's probably a bit above what I have capacity to do rn, happy if you want to pick it up and try it! cc. @ImMattDavison |
Hey @gavinroderick there is already a PR for this, it just needs rearranging so that MV2 and MV3 versions are separated, I can work on it over the weekend unless you want to? |
yeah I looked at the PR, I'll be honest I don't have any experience with extensions and figured this would be a nice place to start! works been super draining this week so i don't think i'll be working on it, sorry! |
That's fair enough, thanks for your help! |
Late here, but I would love to pick up on the issue. @ImMattDavison can I still work on this? |
For sure - please not FireFox requires MV2 - the best option would be to put all current files into a folder called MV3 and make another version with the same functions in a folder named MV2 of course leave README and other similar files in the root dir |
@codingfarhan #61 may be a good starting point! |
Thanks, I'll get right into it. |
Thanks! Looking forward to your PR! |
Facing difficulty implementing MV2 for Firefox, I may not be able to do it. Sorry guys. Please look over it @ImMattDavison |
create a conversion of this extension for mozilla firefox
The text was updated successfully, but these errors were encountered: