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

shortcuts work except for next song #3

Closed
cereal2nd opened this issue Nov 27, 2017 · 17 comments
Closed

shortcuts work except for next song #3

cereal2nd opened this issue Nov 27, 2017 · 17 comments

Comments

@cereal2nd
Copy link

all shortcuts seem to work, except ctrl+shift+0 this one is not responding

@lidel
Copy link
Owner

lidel commented Nov 27, 2017

Hi, I was unable to reproduce, need more information from you:

  • What is the version of your browser?
  • Could you provide URL at which the issue occurs?

@cereal2nd
Copy link
Author

firefox version: 58.0b4 (64-bit)

what do you mean the url?
this is the google play music url: https://play.google.com/music/listen?u=0#/home

Note i also have the containers plugin running

@lidel
Copy link
Owner

lidel commented Nov 27, 2017

Thanks, I'll try to replicate it with Firefox Beta later today.

The URL is different depending on which Google Music Page you are on.
For example an album has a dedicated hash so that you can bookmark it.

Are you experiencing the problem for both URLs below?

@cereal2nd
Copy link
Author

cereal2nd commented Nov 27, 2017 via email

@lidel
Copy link
Owner

lidel commented Nov 27, 2017

Did my best, but I was unable to replicate under v58.0b6 with containers plugin.
Tried on two machines (one Windows, one Linux).

Are you running any other extensions in Firefox? What is your operating system?
I suspect something is hijacking ctrl+shift+0 on your machine.

@lidel
Copy link
Owner

lidel commented Dec 13, 2017

@cereal2nd I assume the issue is gone? If not, please reopen with more info.

@lidel lidel closed this as completed Dec 13, 2017
@cereal2nd
Copy link
Author

cereal2nd commented Dec 14, 2017 via email

@lidel lidel reopened this Dec 14, 2017
@harpratap
Copy link

I'm having the same issue. Firefox 57.0.2 running on Windows 10. Only Ctrl+Shift+0 does not work.

@lidel
Copy link
Owner

lidel commented Dec 16, 2017

Ok, it seems CTRL+SHIFT+0 is hijacked by.. MS Windows (are you both using it?).
The shortcut lets you switch the input language.

Good news is that you can disable it, see instructions below:

Let me know if it helped.

@harpratap
Copy link

It works on Firefox and Chrome now, thanks!

@lidel
Copy link
Owner

lidel commented Dec 23, 2017

Awesome, glad it helped!

@lidel
Copy link
Owner

lidel commented Dec 24, 2017

I am afraid this workaround is not enough. Even tho I've added this information to extension's description at AMO, users are missing it and giving bad reviews :(

Until #1 is closed we can't customize shortcuts under Firefox. What if we add a second set of shortcuts so that users have at least one working set out-of-the box in MS Windows?

  • What set of keys would be appropriate as a backup?
  • Or maybe just provide a backup alternative for the problematic one (play next)?
    Would CTRL+SHIFT+- work?

@lidel lidel reopened this Dec 24, 2017
@harpratap
Copy link

Yes either that or CTRL + SHIFT + [
Both are good choices

@lidel
Copy link
Owner

lidel commented Dec 26, 2017

Unfortunately shortcuts with [ are not permitted under Firefox:

ERROR Loading extension 'null': Reading manifest: Error processing commands.previous-song-backup.suggested_key.default: Value "Ctrl+Shift+[" must either: match the pattern /^\s*(Alt|Ctrl|Command|MacCtrl)\s*+\s*(Shift\s*+\s*)?([A-Z0-9]|Comma|Period|Home|End|PageUp|PageDown|Space|Insert|Delete|Up|Down|Left|Right)\s*$/, match the pattern /^\s*((Alt|Ctrl|Command|MacCtrl)\s*+\s*)?(Shift\s*+\s*)?(F[1-9]|F1[0-2])\s*$/, or match the pattern /^(MediaNextTrack|MediaPlayPause|MediaPrevTrack|MediaStop)$/

What about ALT+SHIFT+0? Would that work under Windows 10?

@harpratap
Copy link

Sorry, I don't have windows 10 to test this out. Personally, I would prefer CTRL + SHIF + -

@lidel
Copy link
Owner

lidel commented Dec 28, 2017

I've released v1.5.0 with additional set of shortcuts prefixed with Alt+Shift:

  • toggle playback: Ctrl+Shift+8 OR Alt+Shift+8, visual hint: *
  • previous song: Ctrl+Shift+9 OR Alt+Shift+9, visual hint: (
  • next song: Ctrl+Shift+0 OR Alt+Shift+0, visual hint: )

Hopefully this will prove to be flexible enough to work on all platforms until #1 is fixed in Firefox.
Feel free to reopen this issue if none of the above work for you.

@lidel lidel closed this as completed Dec 28, 2017
lidel added a commit that referenced this issue Dec 28, 2017
This should fix #3 under Windows, hopefully.
@lidel
Copy link
Owner

lidel commented May 13, 2018

FYSA just released v1.6.0 which enables users to customize shortcut keys under Firefox (see #1 (comment) for more details).

To make it work I had to do two changes that may impact Windows users:

  1. I removed backup set of shortcuts (you can customize original ones, so there is no need for them)
  2. I changed the default shortcuts under windows to Alt+Shift ones to avoid issue with Ctrl+Shift+0 and make it work out-of-the-box for new users. You can always change them anyway, so it should not be a big problem.

I hope this small shakeup will not annoy anyone too much :)

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