Bug 407651 - Window gets bigger after every closing (and opening) to system tray
Summary: Window gets bigger after every closing (and opening) to system tray
Status: RESOLVED UPSTREAM
Alias: None
Product: plasmashell
Classification: Plasma
Component: generic-wayland (show other bugs)
Version: 5.15.90
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-05-17 16:00 UTC by Matej Mrenica
Modified: 2019-06-26 10:52 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Matej Mrenica 2019-05-17 16:00:33 UTC 
SUMMARY
I tested this with qbittorent

STEPS TO REPRODUCE
1. Open qbittorent
2. Close bittorent to system tray
3. Open bittorent from system tray
4. repeat 2. and 3. 

OBSERVED RESULT
Every time the window is opened it is created little wider than before (it goes more and more beyond screen edge)

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.15.90
KDE Frameworks Version: 5.58
Qt Version: 5.13.0 beta3

ADDITIONAL INFORMATION
On wayland
Comment 1 Matej Mrenica 2019-05-17 16:03:47 UTC 
There is also another way to trigger this:

Open Dolphin a few times, you can see that every new instance has a slightly bigger window.
Comment 2 David Edmundson 2019-05-17 17:12:02 UTC 
I just checked the logs.

Kwin is not specifying a size in the configure event, so the bug is purely client side (the Qt QPA) probably it adjusting as we toggle from CSD.

Needs fixing in Qt. I'll leave this open till we submit a patch.
Comment 3 David Edmundson 2019-05-22 23:52:55 UTC 
Patch: https://codereview.qt-project.org/c/qt/qtwayland/+/262587
Comment 4 Matej Mrenica 2019-06-04 14:34:10 UTC 
Is Qt 5.13.0 rc1 supposed to have fixed this? I am asking because it's still broken.
Comment 5 Christoph Feck 2019-06-25 17:50:23 UTC 
It probably didn't make it to RC1. Please test 5.13.0

https://blog.qt.io/blog/2019/06/19/qt-5-13-released/
Comment 6 Matej Mrenica 2019-06-26 06:29:29 UTC 
(In reply to Christoph Feck from comment #5)
> It probably didn't make it to RC1. Please test 5.13.0
> 
> https://blog.qt.io/blog/2019/06/19/qt-5-13-released/

So, I tested with qt 5.13.0 on Arch and it's still broken.
Comment 7 Matej Mrenica 2019-06-26 06:40:16 UTC 
I also checked the source and the patch isn't there, either it wasn't accepted or it has been reverted.
Comment 8 David Edmundson 2019-06-26 07:02:18 UTC 
Qt branch off 5.13.0 before release day.

It takes special exception to get a change in there.

It should be in 5.13.1
Comment 9 Matej Mrenica 2019-06-26 10:52:00 UTC 
This patch is now included in Arch Linux starting qt5-wayland 5.13.0-2