Don't close full zoom on keydown

[pintassilgo]

In #85 it was implemented the option keepOnBlur. But I'm still annoyed by images disappearing when I press Ctrl+Tab to switch active tab or Alt+Tab to switch active window. I'm forced to use the mouse in these cases.

So I suggest an option to not close full zoom on keydown, as I'm used to do with Imagus.

Or at least use evt.ctrlKey || evt.altKey || evt.metaKey as exception to not close full zoom. Or evt.key.length > 1...

[tophf]

Ctrl was always a key to close the preview in MPIV. The problem is that Tab in these context-switching keys isn't reported to JS by the browser, but I think I can switch to the keyup event instead of keydown and check document.hasFocus()...

As for Alt-Tab, it doesn't close the preview for me so I need more info on that.

[pintassilgo]

Ctrl was always a key to close the preview in MPIV.

Curious, I think Esc is much more suitable for that action.

The problem is that Tab in these context-switching keys isn't reported to JS by the browser,

Right now, what is closing the preview is Ctrl (in case of Ctrl+Tab) and Alt (in case of Alt+Tab). So Tab doesn't need to be tracked.

As for Alt-Tab, it doesn't close the preview for me so I need more info on that.

It's Alt, but it's inconsistent. Sometimes it hides the preview, sometimes it does nothing.
It is worth saying that I use Firefox and Alt activates its menubar.

[tophf]

The problem is that I want to track Tab to preserve the historic behavior of closing the preview via Ctrl. I can't reproduce the Alt-Tab problem in FF either. I wonder if showing the menu simply shifts the page so the cursor leaves the area of the thumbnail?

[pintassilgo]

The problem is that I want to track Tab to preserve the historic behavior of closing the preview via Ctrl.

I know you avoid adding user options, but this would be a good way to fix. Which key to close the preview, defaulting Ctrl but allowing to choose Esc.

I wonder if showing the menu simply shifts the page so the cursor leaves the area of the thumbnail?

It can't be. When I say "full zoom" (I activate it by right-click), I'm not talking about "preview on hover". Cursor position can't close image in full zoom mode.

[tophf]

Since this is the first time I personally encounter such a request I doubt an option is warranted.

In full zoom mode the image isn't guaranteed to fill the entire viewport depending on its proportions so if the cursor was near the edge of the preview it may theoretically move out of the area when the Alt menu is shown.

[tophf]

I found a consistent repro in FF: switching to a full-screen app via Alt-Tab. Since it doesn't happen in Chrome I guess it might be a bug in FF. I'll see if I can circumvent it...

[pintassilgo]

In full zoom mode the image isn't guaranteed to fill the entire viewport depending on its proportions so if the cursor was near the edge of the preview it may theoretically move out of the area when the Alt menu is shown.

I'm not getting what you're saying. In full zoom mode, image is centered and I can move cursor anywhere, moving cursor can't close an image in full zoom mode.

I found a consistent repro in FF: switching to a full-screen app via Alt-Tab.

Just saying I don't use full-screen. Actually, my use case is that I have Firefox tiled with a text editor and I need to transcribe text within image.

First I hover the image, then I right-click it to enter in full-zoom mode, because letters are small so I need to see the image in real size.

Then I press Alt+Tab to focus text editor, so that I can write the text I'm reading from the image. But pressing Alt is causing the image to close...