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

Erratic behavior of F11 and F12 #885

Closed
skycommand opened this issue Oct 8, 2020 · 4 comments
Closed

Erratic behavior of F11 and F12 #885

skycommand opened this issue Oct 8, 2020 · 4 comments

Comments

@skycommand
Copy link

skycommand commented Oct 8, 2020

I've updated this issue to reflect changes in version 8.0.12.8; I decided that updating this topic is better than opening a new issue because the nature of the problem hasn't changed. (Past versions of this message are also available.)

System information:

  • Windows OS version: 10.0.19042.746 (also known as 20H2)
  • ImageGlass version: 8.0.12.8

To Reproduce

Steps to reproduce the behavior:

  1. Prepare a group of images in a folder
  2. Enter the folder
  3. Double-click the first image to launch ImageGlass
  4. Press F11 to enter the full-screen mode
  5. Press F12 to enter the slideshow mode

Expected behavior:

Step 5 must cause the slideshow to start in chromeless full-screen mode

Actual behavior:

Image glass enters the windowed mode before starting the slideshow.

From this point onward, F12 and F11 behave erratically. Sometimes F11 activates the fullscreen mode and other times, it just reduces the window size to a portion of the screen. The F12 key may or may not trigger the slideshow in chromeless full-screen mode.

Screenshots:

N/A

Additional context:

N/A

@skycommand skycommand changed the title Erratic behavior of F11: Sometimes, it reveals the thumbnails panel instead of starting slideshow Erratic behavior of F11: Sometimes, instead of starting slideshow, it does something else Oct 9, 2020
@d2phap d2phap added this to the Issues 7.6.4.30 milestone Oct 11, 2020
@d2phap d2phap removed this from the Issues 7.6.4.30 milestone Nov 18, 2020
@StevenLOL StevenLOL mentioned this issue Jan 4, 2021
@skycommand skycommand changed the title Erratic behavior of F11: Sometimes, instead of starting slideshow, it does something else Erratic behavior of F11 and F12 Jan 18, 2021
@d2phap
Copy link
Owner

d2phap commented Mar 7, 2021

Just a note to look back in the future:
This is because IG does not support Slideshow mode from startup.

@skycommand
Copy link
Author

This is because IG does not support Slideshow mode from startup.

If that was true, I should have been able to work around the problem by waiting for a few seconds. This doesn't seem to be the case.

Anyway, I can't see how that is relevant. Erratic behaviors like this go against the principle of the least astonishment. For a well-organized mind that writes well-organized code, this mustn't be a problem. All you have to do is to approach the subject systematically, by separation of concerns. For instance, you can:

  1. Assign all UI entry points of the slideshow (toolbar, menu, keyboard, etc.) to a method that invokes the slideshow at the load time
  2. Keep track of component loading status via a private boolean field (especially, if the app is multi-threaded)
  3. Have the slideshow method check the boolean field before doing anything

@fire-eggs
Copy link
Collaborator

"Start to Fullscreen to Slideshow" goes into windowed mode because the FullScreenMode function always turns off frameless mode, even in this case where we don't want to.

I believe the subsequent "confusion" of F11 and F12 could then be due to the disconnect between the IsFullScreen flag and the actual window state. I.e. after the sequence "Start to Fullscreen to Slideshow", the IsFullScreen flag is ON but IG is actually windowed.

I think it may be necessary for FullScreen / WindowFit / Frameless to be a little smarter about SlideShow [and visa-versa] but I've not worked out all the implications.

@d2phap
Copy link
Owner

d2phap commented Dec 24, 2022

Slideshow mode is supported in ImageGlass Moon 9.0 beta 3 with full screen / windowed mode

@d2phap d2phap closed this as completed Dec 24, 2022
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

3 participants