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

Google image search: search box should not be focused when closing preview with escape #161

Open
infokiller opened this issue Jun 22, 2020 · 4 comments

Comments

@infokiller
Copy link
Owner

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)

@infokiller infokiller changed the title Google image search: escape behavior in image preview should close preview Google image search: search box should not be focused when closing preview with escape Jun 22, 2020
@infokiller
Copy link
Owner Author

@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).
However, fixing the opening of the search box with escape will probably not go in to the next release and may take more time to fix. This issue will track progress.

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.

@infokiller infokiller added the bug label Jun 22, 2020
@eggbean
Copy link

eggbean commented Jun 22, 2020

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 😃

@eggbean
Copy link

eggbean commented Jun 22, 2020

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.

@infokiller
Copy link
Owner Author

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.

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

No branches or pull requests

2 participants