-
Notifications
You must be signed in to change notification settings - Fork 42
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
Some hotkeys still dont work #68
Comments
@brianteeman actually I've added help text to all missing sections (version |
Which browser are you using? I still can't get ctr alt a to work for me AND it wasnt on the documented list but I see that invert colors has a hotkey as well (ctrl alt i) and that one dosesnt seem to work either |
found it - they must have been conflicting with another hotkey - I changed them to y and n and it worked Maybe you know of a tool to check what is being triggered by a hotkey |
Not really but I've merged your PR, thank you for that. version 5.0.6 has been published accordingly. |
Sorry if there is any confusion - when I said " I changed them to y and n and it worked" that was just for testing to make sure it was a key conflict. That was not in the pr - just some other missed stuff. |
I'm using Chrome on Windows. Just tried |
obviously a conflict with something I have installed on my pc then |
retested on chrome without any extensions and still have the issues reported in the first post with some hotkeys not working |
You'll need to be more specific on how to reproduce this issue. WhatsApp.Video.2024-06-15.at.10.57.25.mp4 |
This recording was made with chrome 125 on windows 11. Running in incognito mode so no browser extensions are interfering. Keypresses are displayed using keyviz - the same happens without that application running - it was just for display |
@brianteeman according to the video I see that all of the hotkeys doesn't work. I was under the impression that most of them are working by looking at the table at the top comment. |
I think the video is not clear. I will try again tomorrow |
Verified on your demo site
The text was updated successfully, but these errors were encountered: