-
Notifications
You must be signed in to change notification settings - Fork 72
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
Keyboard shortcuts don't work consistently in the images tab #133
Comments
I tested this in Firefox and it actually worked. |
For me, keyboard shortcuts work fine in images tab during incognito mode (with the rest of my extensions also enabled). However, during a regular session (non-incognito mode), the keyboard shortcuts do not work in the images tab even with all extensions but web-search-navigator disabled. |
Can't speak for all shortcuts, but I think I know why the 'focus search box' shortcut is broken. It assumes the search box is a DOM element with the selector (don't judge; I was looking for a very important GIF) Google probably started uglifying class names recently, possibly as some sort of guard against reverse-engineering the page like we're doing here 😬 If that new class name remains constant, we can fix this by simply hardcoding it in. Or maybe we can select the search box with |
Thanks for your input! |
For me, just going to the images tab makes the whole extension totally unusable :( I can hop between the search tab, the videos tab, and all other tabs just fine, but as soon as I go to the images tab I can't regain focus of the navbar or switch to any other tab. I tested this in normal chrome with only this extension. |
@williamfiset till this is fixed, you can work around that by using a keyboard shortcut for back. I use a private unreleased extension for this to set the keyboard shortcuts for back/forward (can release it if people are interested), but there's probably other extensions around for that. |
Thanks for the tip, I think in Chrome you can use ⌘ + ← to return to the previous page, at which point you can continue using the extension normally. This is a manageable workaround since it doesn't require me to use my mouse, but I feel like a proper fix is still warranted. |
Really great that this is working as well now. But if I can suggest what I think would be a minor improvement: When you highlight a picture and press ENTER, the preview pane opens on the right. Once the preview pane is closed again using ESC, instead of the previous thumbnail being selected (which would be preferable), the search text entry box is. I think it would also be nice if the selected thumbnail bold outline highlighting was a little thicker, so it's easier to see. |
Yes @eggbean I agree the search box should not be selected in this case. I will try to fix it. |
No description provided.
The text was updated successfully, but these errors were encountered: