Review Request 124869: Fix widget explorer sometimes opening on the wrong side of the screen.

Martin Gräßlin mgraesslin at kde.org
Mon Aug 24 06:35:06 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124869/#review84243
-----------------------------------------------------------



desktoppackage/contents/views/Desktop.qml (line 87)
<https://git.reviewboard.kde.org/r/124869/#comment58338>

    please add a FIXME: remove once we depend on fixed Qt version


- Martin Gräßlin


On Aug. 21, 2015, 7:12 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124869/
> -----------------------------------------------------------
> 
> (Updated Aug. 21, 2015, 7:12 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Work around a Qt bug.
> 
> If there is no setGeometry call on QWindow the XCB backend will not pass
> our requested position to kwin
> as our window position tends to be 0, setX,setY no-ops and means we
> never set a position as far as QWindow is concerned
> by setting it to something silly and setting it back before we show the
> window we avoid that bug.
> 
> Qt patch also filed (https://codereview.qt-project.org/123989)
> so hopefully this can be removed in future if that gets merged.
> 
> BUG: 332702
> 
> 
> Diffs
> -----
> 
>   desktoppackage/contents/views/Desktop.qml e52108c7f2a4fd58a74642d09893c51fbf701f55 
> 
> Diff: https://git.reviewboard.kde.org/r/124869/diff/
> 
> 
> Testing
> -------
> 
> Had a billion lines of qDebug in kwin that shows it now gets a geometry in KWin::manage.
> 
> Also window is always in the right place \o/
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150824/c701c5a7/attachment.html>


More information about the Plasma-devel mailing list