-
Notifications
You must be signed in to change notification settings - Fork 75
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
Google image search: search box should not be focused when closing preview with escape #161
Comments
@eggbean I made the outline thicker and added results navigation (which means you can configure the keybindings in the extension options as an alternative to Google's keybindings). As a workaround, you should be able to just leave the preview pane open (don't close it with escape) and keep navigating the results. |
Yes, I seem to have this update already, so I suppose it must be server based. I have just noticed that once I remove ESC as the alternative shortcut to focus on the search box (I only use '/'), this behaviour stops, and now the extension works pretty much perfectly as I would like 😃 |
Actually. there are issues. Sometimes there are two types of highlighted thumbnail. There is the black outlined one, from the extension, but when the preview pane is opened, the selected and highlighted thumbnail is a blue shaded box (preferable aesthetically). When the preview pane is closed with ESC, then both both highlighted boxes often remain, both independent of each other. Also, it is not the last previewed thumbnail which is selected with the black outline - it's the first image instead (this is after disabling ESC as 'search box focus'). The only keys that work in the preview pane mode are left and right arrow, but I think this is okay, as the top right corner left/right icons are what I would like to be using with the keys anyway. |
Yes, this issue will be slightly complex to fix in a clean way. My suggested workaround is to always navigate with the extension keys (up/down and j/k by default), and if you want to navigate when the preview pane is open, navigate to the image you want with the extension keys (which will only move the black outline, the shaded blue with not move), then press space/enter to load it to the preview. |
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.
Originally posted by @eggbean in #133 (comment)
The text was updated successfully, but these errors were encountered: