Bug 354741 - Cancelling quick tile causes window to flicker
Summary: Cancelling quick tile causes window to flicker
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: effects-various (show other bugs)
Version: git master
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords:
: 401635 488230 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-11-02 17:05 UTC by Kai Uwe Broulik
Modified: 2024-06-10 18:22 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.1.0
Sentry Crash Report:


Attachments
Support information (5.53 KB, text/plain)
2015-11-02 17:05 UTC, Kai Uwe Broulik
Details
Video (3.76 MB, video/ogg)
2015-11-02 17:07 UTC, Kai Uwe Broulik
Details
Flicker in 5.27.5 (843.10 KB, video/mp4)
2023-06-16 21:47 UTC, cairn overturf
Details
reupload (1.31 MB, image/gif)
2023-06-16 21:53 UTC, cairn overturf
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kai Uwe Broulik 2015-11-02 17:05:46 UTC
When dragging a window to the top edge of the screen, the quick tile frame appears. When moving the window away from the corner again, it disappears but the window briefly flickers (looks like it hides and fades back in again).

Might be some stacking issue since when I move the window with Alt+left mouse (where I can move it behind the panel) it is rendered above the panel while the quick tile frame is shown.

Reproducible: Always

Steps to Reproduce:
1. Move window to the top corner of the screen (or top left, or any other corner that triggesr quick tiling)
2. Move the window away from the corner without releasing the mouse button
3.

Actual Results:  
The window disappears and fades back in briefly

Expected Results:  
The window stays as it was
Comment 1 Kai Uwe Broulik 2015-11-02 17:05:57 UTC
Created attachment 95267 [details]
Support information
Comment 2 Kai Uwe Broulik 2015-11-02 17:07:01 UTC
Created attachment 95268 [details]
Video
Comment 3 Kai Uwe Broulik 2015-11-02 17:07:57 UTC
It also happens on XRender, OpenGL 2, OpenGL3 with both EGL and GLX
Comment 4 Thomas Lübking 2015-11-02 22:51:22 UTC
The window eelvation is broken on the cancel, the overlay indication thus on top while it fades out.
Comment 5 Vlad Zahorodnii 2018-08-18 20:29:05 UTC
Ping?
Comment 6 Vlad Zahorodnii 2018-08-21 21:03:46 UTC
So, the root of the problem is the Fade effect. The outline window should not be animated by any of current effects. (if we want it to fade in/out, then we have to have dedicated effect for that)

On X11, that's not really easy to fix this bug.

On Wayland, things are a little bit better. We can lower the outline window. So, during fade out animation, it will stay below its "parent" window.

PoC: https://github.com/zzag/kwin/commit/0c56c1febf543ae228cea556119ddfe805be2d0f
Comment 7 Vlad Zahorodnii 2018-08-21 21:16:07 UTC
Please notice this is just a PoC. Adding internal clients to stacking orderis probably not the greatest thing to do.
Comment 8 Vlad Zahorodnii 2018-09-22 11:25:33 UTC
Here's potential solution:

* We need to have an explicit class for internal clients, e.g. Internal;
* That class should allow setting anchors, e.g.

   enum class Relationship {
       Below,
       Above
   }

   Toplevel *Internal::anchor() const;
   Relationship Internal::anchorRelationship() const;
   void Internal::setAnchor(Toplevel *toplevel, Relationship relationship);

   (... or maybe just use transientFor?)

* Also, maybe, extend window types (so we can have a dedicated effect that animates the appearing/disappearing of outline), e.g.

   bool EffectWindow::isOutline() const;

   void FadeOutlineEffect::slotWindowAdded(EffectWindow *w)
   {
       if (!w->isOutline()) {
           return;
       }

       // Setup animation data.
   }

   void FadeOutlineEffect::slotWindowClosed(EffectWindow *w)
   {
       if (!w->isOutline()) {
           return;
       }

       // Setup animation data.
   }

* In layers.cpp, after we've got the constrained stacking order, we append internal clients to it [to the constrained stacking order] and then do something similar to what we do with transients in constrainedStackingOrder:

    - if Internal::anchor() returns nullptr, move to the next internal client;
    - otherwise, find position of the anchor in the stacking order;
    - if position is equal to -1, move to the next internal client;
    - otherwise, move the internal client to that position depending on the relationship.

* Add to the Outline class a show method that receives an AbstractClient and also maybe relationship (should the outline be above or below the client). Composited outline would keep track of outline for each window. With compositing being off, the anchor can be ignored, I guess;

* Adjust AbstractClient::setElectricBorderMaximizing, so it passes itself as the anchor.


.... that's just an idea.
Comment 9 Vlad Zahorodnii 2018-12-02 13:16:38 UTC
*** Bug 401635 has been marked as a duplicate of this bug. ***
Comment 10 postix 2022-03-04 13:24:31 UTC
I see that also on KWin 5.24.2 Wayland.
Comment 11 postix 2023-06-16 10:05:17 UTC
I don't see that anymore under Plasma 5.27.5 on Wayland. Can you still reproduce it Kai?
Comment 12 cairn overturf 2023-06-16 21:47:06 UTC
I still see this issue:

Operating System: Fedora Linux 38
KDE Plasma Version: 5.27.5
KDE Frameworks Version: 5.107.0
Qt Version: 5.15.10
Kernel Version: 6.3.7-200.fc38.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 8 × AMD Ryzen 5 3550H with Radeon Vega Mobile Gfx
Memory: 13.6 GiB of RAM
Graphics Processor: AMD Radeon Vega 8 Graphics
Manufacturer: BESSTAR TECH LIMITED
Product Name: UM350
Comment 13 cairn overturf 2023-06-16 21:47:56 UTC
Created attachment 159724 [details]
Flicker in 5.27.5
Comment 14 cairn overturf 2023-06-16 21:53:40 UTC
Created attachment 159725 [details]
reupload

Here's a gif because the site was saying the mp4 is corrupt
Comment 15 Bug Janitor Service 2023-07-01 03:45:09 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 16 Zamundaaa 2024-06-07 13:25:15 UTC
Git commit b2ff46549075343710bc0e106b9e80085eae7823 by Xaver Hugl.
Committed on 07/06/2024 at 12:46.
Pushed by zamundaaa into branch 'master'.

workspace,window: don't elevate windows during electric border tiling / maximizing

Instead, move the outline below the window, so that the visual order of things stays the
same. This also fixes a visual glitch, where the outline is visible for a moment after
maximization, and is above the no-longer-elevated window
Related: bug 436466

M  +3    -2    src/window.cpp
M  +6    -0    src/workspace.cpp

https://invent.kde.org/plasma/kwin/-/commit/b2ff46549075343710bc0e106b9e80085eae7823
Comment 17 Zamundaaa 2024-06-07 13:38:13 UTC
Git commit c0ca01317594dbbaa6fd8c8d86c79c86886b15c2 by Xaver Hugl.
Committed on 07/06/2024 at 13:27.
Pushed by zamundaaa into branch 'Plasma/6.1'.

workspace,window: don't elevate windows during electric border tiling / maximizing

Instead, move the outline below the window, so that the visual order of things stays the
same. This also fixes a visual glitch, where the outline is visible for a moment after
maximization, and is above the no-longer-elevated window
Related: bug 436466


(cherry picked from commit b2ff46549075343710bc0e106b9e80085eae7823)

Co-authored-by: Xaver Hugl <xaver.hugl@gmail.com>

M  +3    -2    src/window.cpp
M  +6    -0    src/workspace.cpp

https://invent.kde.org/plasma/kwin/-/commit/c0ca01317594dbbaa6fd8c8d86c79c86886b15c2
Comment 18 Zamundaaa 2024-06-10 18:17:37 UTC
*** Bug 488230 has been marked as a duplicate of this bug. ***