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? #48

Open
wldcordeiro opened this issue Feb 6, 2018 · 7 comments
Open

Firefox support? #48

wldcordeiro opened this issue Feb 6, 2018 · 7 comments
Assignees

Comments

@wldcordeiro
Copy link

Now that Firefox uses webextensions this really should be easier to maintain for both browsers.

@ducky
Copy link
Owner

ducky commented Feb 6, 2018

Yeah it's definitely on my list. It's mostly 80%-ish the way there, but there were some issues that I still need to look into that I mentioned in a comment on this PR: #42

@ducky ducky self-assigned this Feb 6, 2018
@phasze
Copy link

phasze commented Mar 23, 2020

Any update on this?

@ducky
Copy link
Owner

ducky commented Mar 24, 2020

@phasze - Sadly no. I’ve been busy with a few other projects so I’ve mainly been in maintenance mode with Play Midnight the last year or two. I’m always open to other people PRing it if they wanna tinker.

@phasze
Copy link

phasze commented Mar 24, 2020

Alright I'll take a look today. I swapped to firefox recently because of some of chrome's policy changes and have been missing your lovely extension.

I built your project and loaded it as a firefox addon. Working great so far. I see the PR you linked had a few issues, are those still the outstanding ones? Anything else I should look out for?

@ducky
Copy link
Owner

ducky commented Mar 24, 2020

Thanks for looking into it. As far as I know those are the main issues. I'll be honest I haven't looked at Play Music (especially Firefox) in ages since I use Spotify as my main driver now.

I know the biggest issue was the first one I mentioned which was saving. Rather than adopt what Chrome has working with callbacks, Firefox opted to use promises which made working them together a pain.

I can't say if those three issues are the only ones anymore, so I'm all ears, haha.

@phasze
Copy link

phasze commented Mar 24, 2020

I don't see the homepage scrolling issue.

The favicon swapping does appear to still be an issue. From a quick glance at the attribute in the inspector I suspect this is because the file doesn't exist in the location it tries to set.

As for saving, I assume this is talking about saving the user preferences when they click on the gear icon and change something? From my quick test that appears to be working as well.

@ducky
Copy link
Owner

ducky commented Mar 24, 2020

  • Scrolling: Cool, I don't even remember what the actual issue is anymore.
  • Favicon: Yeah, I think it was just a disconnect to where the background pages put things. Most likely the way the url is referenced. I never looked much into it.
  • Saving: I believe it's the sync settings. You'd have to check it between two firefox browsers (two different machines) to make sure the settings are synced across all browsers. I think I use local storage as a backup, maybe?

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

No branches or pull requests

3 participants