You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is different to window visibility as it depends on whether the window is closed, minimised, set invisible, or fully occluded by another window.
Window::is_visible() says:
Gets the window's current visibility state.
Even though WindowEvent::Occluded differentiates itself from "window visibility", "window visibility" itself is not really explained anywhere. Probably the mention of "set invisible" in Occluded doesn't help either.
The text was updated successfully, but these errors were encountered:
There was a brief discussion in IRC about adding a Window::is_occluded(), but I'm really not sure this makes sense because of the ambiguity around "Window::is_visible()`.
Additionally with #2929, we have another ambiguity: WindowState, if the window is minimized, is it also Occluded? I'm just not sure if it makes sense to split up like that.
Just some quick not really thought through suggestion:
Maybe WindowState should also include "window visibility" and maybe we should also remove is_maximized() and all these other methods and just expose Window::window_state(), then we could have one event: WindowEvent::Configured that handles all states and one method Window::window_state() that can query all of them as well (maybe Window::set_window_state() as well?).
The WindowEvent::Occluded event should always fire if the window isn't visible, and with visible I mean if the user can see the window or not, period. Then a Window::is_occluded() makes sense and would also be true if minimized or doesn't have "window visibility".
WindowEvent::Occluded
says:Window::is_visible()
says:Even though
WindowEvent::Occluded
differentiates itself from "window visibility", "window visibility" itself is not really explained anywhere. Probably the mention of "set invisible" inOccluded
doesn't help either.The text was updated successfully, but these errors were encountered: