Skip to content
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

firefox support #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ShadowVariable
Copy link

Adds support for firefox.

@ShadowVariable ShadowVariable mentioned this pull request Nov 25, 2017
@mimen
Copy link
Owner

mimen commented Nov 26, 2017

@ShadowVariable I'm having trouble getting the firefox extension packaged and running. Were you able to have success? Do you have time to hop on some messenger and help me figure it out? From what I can tell, we can use the same script and manifest for both chrome and firefox, but I'm having some compatibility issues with my firefox browser.

@ryan-mccaffrey
Copy link

@ShadowVariable I just realized I made the same PR as you, but mine should work. I believe the issue with yours is that the instances of "chrome" need to be changed to "browser" in the script.js file for firefox

@ShadowVariable
Copy link
Author

@ryan-mccaffrey I was reading through the Firefox documentation and it said that "chrome" is automatically changed to "browser" when loaded in firefox which is why I didn't change it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants