Commits on Source (20)
-
When we're moving a view to the minimised layer, use our new(ish) helper to move it. Signed-off-by: Daniel Stone <daniels@collabora.com>
8ed7f0f4 -
weston_shell_utils_center_on_output() already calculates the subsurface bounding box for us. Signed-off-by: Daniel Stone <daniels@collabora.com>
2ab3d339 -
weston_shell_utils_center_on_output() already handles the case where we don't have an output to aim at. Signed-off-by: Daniel Stone <daniels@collabora.com>
e2a48f6f -
Use our helper to move stuff around. Signed-off-by: Daniel Stone <daniels@collabora.com>
4bbe4e6e -
If we're asked to set a view as fullscreen, assume we're not doing so with the surface not being mapped. Signed-off-by: Daniel Stone <daniels@collabora.com>
49a122bd -
shell_map_fullscreen() just duplicated shell_configure_fullscreen() exactly. Signed-off-by: Daniel Stone <daniels@collabora.com>
e4e3cc19 -
Merge shell_configure_fullscreen() and shell_ensure_fullscreen_black_view() into shell_set_view_fullscreen(). Signed-off-by: Daniel Stone <daniels@collabora.com>
a019aa6b -
The only place we ever set shsurf->fullscreen.black_view is when we're going into fullscreen, and we destroy it when we're going out of fullscreen. Hence if we do ever have a fullscreen black view, we should move it out of the way. Signed-off-by: Daniel Stone <daniels@collabora.com>
e3255713 -
However many times you thought this might reasonably be called when setting up for fullscreen, there were more. Signed-off-by: Daniel Stone <daniels@collabora.com>
77b9dbb7 -
We must have (from || to), because if (!from && !to), we would've caught the if (from == to) early-return case. Signed-off-by: Daniel Stone <daniels@collabora.com>
db978209 -
Rather than juggling our layers manually, just use weston_view_move_to_layer. This obviates the need for manually marking the view as mapped. Signed-off-by: Daniel Stone <daniels@collabora.com>
6738c738 -
We don't need to be setting this because it isn't (yet) used anywhere. It will get set when it is used. Signed-off-by: Daniel Stone <daniels@collabora.com>
a168dc06 -
Rather than creating them the first time we need to switch focus, just create them at startup if we're going to be needing them at some point. Signed-off-by: Daniel Stone <daniels@collabora.com>
42334db4 -
Now that we're not allocating surfaces on demand, animate_focus_change() becomes a lot more straightforward and common, and using some local variables sure does cut out a lot of typing. Knowing that both from and to cannot both be NULL (because we check if from == to), we can change the juggling to be extremely simple: calculate where we need the curtains to be in the view list, put them there, and set up the fade. Signed-off-by: Daniel Stone <daniels@collabora.com>
f04e1ec9 -
Make sure that we can exit cleanly if we have to fail. Signed-off-by: Daniel Stone <daniels@collabora.com>
fca71ff4 -
Just refuse to start up if someone passes an incorrect focus type. Signed-off-by: Daniel Stone <daniels@collabora.com>
72a52454 -
Signed-off-by: Daniel Stone <daniels@collabora.com>
f17581fd -
It already does what we want, so just use it when we're losing focus. Signed-off-by: Daniel Stone <daniels@collabora.com>
73050c35 -
We don't need to inflict any damage here, because it's been done for us when we move the layers. Signed-off-by: Daniel Stone <daniels@collabora.com>
fb64b00d -
Use our new helper instead of doing this by hand. Signed-off-by: Daniel Stone <daniels@collabora.com>
f7d80953