Commits on Source (5)
-
weston_view_move_to_layer() recalculates the output, so this can do no good. This output is only ever used to test against NULL anyway, so we don't need to try to set it "properly". Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
d215dc83 -
We've already validated this, so if we can't get here with an invalid type. Just assert() instead. This lets us remove the very strange tear down of curtains that presumably couldn't have been created in the first place?? Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
59b6e697 -
This essentially reverts 9ad4de1f, though much has changed in the interim to make a simple 'git revert' impossible. Per-output fade animations were a precursor to implementing support for zwp_idle_inhibit_v1. Nobody ever followed up on that. Now that we have the ability to create overlapping outputs, per output curtains don't make sense - the curtains can overlap and look weird. This re-introduces the bug where a desktop larger than 8192 pixels in any direction isn't properly covered by the fade. Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
06930c71 -
Instead of trying to cover the desktop with an arbitrarily larger rectangle, actually calculate the appropriate size. Fixes the old (but recently reintroduced) bug where the curtain isn't always large enough. Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
dd956e5b -
This is a trap - it's not harmful to let the animation complete - it is harmful to call weston_shell_utils_curtain_destroy() in mid animation. It results in immediately completing the animation, which attempts to destroy the curtain a second time. Fixes a crash when (on a multi-output system) the display is disconnected during the fade out, then reconnected before the fade-in starts. Signed-off-by: Derek Foreman <derek.foreman@collabora.com>
8ac621d6