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

We should make sure other listeners don't get triggered by keypresses when the _hold key_ is down. #35

Closed
carbon-steel opened this issue Jun 5, 2017 · 4 comments
Assignees
Labels

Comments

@carbon-steel
Copy link
Collaborator

Some websites like google drive and gmail use keypresses as commands. We should make sure that keypresses from our hotkeys do not trigger these commands when the holdkey is held down.

@jchang504
Copy link
Owner

Have you found a case where this doesn't work? I'm pretty sure I've already implemented this correctly.

@jchang504 jchang504 modified the milestone: v1.0.0 Release Ready Jun 7, 2017
@carbon-steel
Copy link
Collaborator Author

In Google Drive (when you're logged in), using + f doesn't activate the hotkey, but opens a dropdown on the page instead. Also, + / doesn't open a search tabs page but just makes the page focus on the search query at the top.

Similar things happen on Gmail (if you have your Gmail hotkeys activated in your settings).

@jchang504
Copy link
Owner

jchang504 commented Jun 10, 2017

I can't reproduce those two on https://drive.google.com/drive/my-drive and I've been using my keyboard shortcuts on Gmail fine. Are you sure you weren't working on pages that hadn't been refreshed since you reloaded KeepTabs for some reason? Were KeepTabs hotkeys working at all on those pages (even ones that don't overlap)? If not, you can open up the console and confirm that the pages haven't been refreshed since a KT reload if you see runtime errors talking about runtime.connect and stuff like that.

@jchang504
Copy link
Owner

I just tested this on Mac Chrome with all 4 hold key options (context menu key doesn't exist on Mac) and KeepTabs works as expected on Gmail and Drive. The keyboard shortcuts for those sites don't activate when the hold key is pressed and they do when it's not. I think something else must have been going on when you observed this (e.g. what I mentioned above). Closing this until it can be reliably reproduced.

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

No branches or pull requests

2 participants