Review Request 127409: Make sure we're not forcing the window to be created prematurely
Martin Gräßlin
mgraesslin at kde.org
Thu Mar 17 06:52:51 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127409/#review93633
-----------------------------------------------------------
Did you also check that on Wayland? There it is important to have certain things happen before the window is shown, e.g. the role should be set before the window is shown, but the platform window needs to be set already.
Best way might be to completely integrate with the PlatformSurface QEvent and track whether there is a native window. That way we would not need to rely on visible and force creation through winId.
- Martin Gräßlin
On March 17, 2016, 3:13 a.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127409/
> -----------------------------------------------------------
>
> (Updated March 17, 2016, 3:13 a.m.)
>
>
> Review request for KDE Frameworks and Plasma.
>
>
> Repository: plasma-framework
>
>
> Description
> -------
>
> Only update the properties that require a winId when visible, otherwise let updateVisibility() take care about them.
>
>
> Diffs
> -----
>
> src/plasmaquick/dialog.cpp 7bb63c5
>
> Diff: https://git.reviewboard.kde.org/r/127409/diff/
>
>
> Testing
> -------
>
> I don't get platform creations being forced on gdb.
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160317/88fed15f/attachment.html>
More information about the Kde-frameworks-devel
mailing list