Review Request 119594: fix FileDialog size restorage
Thomas Lübking
thomas.luebking at gmail.com
Mon Aug 4 21:54:23 BST 2014
> On Aug. 4, 2014, 12:11 nachm., Martin Klapetek wrote:
> > This does fix the issue here; +1 from me
> >
> > Also, are you investigating/reporting the bug to Qt devs?
>
> Thomas Lübking wrote:
> On a quick glimpse, it looks like the issue is that QWidgetWindow syncs stuff and that it syncs (only) in ::event() ie. the eventloop must be up for QWindow::setGeometry() to impact QWidget::geometry()
>
> -> I fear that's out of scope for a noon break ;-)
> I'll try to have a more detailed look this week.
https://bugreports.qt-project.org/browse/QTBUG-40584
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119594/#review63765
-----------------------------------------------------------
On Aug. 4, 2014, 11:57 vorm., Thomas Lübking wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119594/
> -----------------------------------------------------------
>
> (Updated Aug. 4, 2014, 11:57 vorm.)
>
>
> Review request for KDE Frameworks, kdelibs, Aleix Pol Gonzalez, Lukáš Tinkl, and Martin Klapetek.
>
>
> Repository: frameworkintegration
>
>
> Description
> -------
>
> - saving in the deconstrutor is insufficient,
> the dialog might survive the runtime
> -> needs to be saved when the dialog is finished or just closed
> (the closeEvent is not invoked if at least a sync dialog is finished)
>
> - ensure a windowHandle and then restore the window size before calling ::exec()
>
> - workaround an apparent QWidget QPA bug where even for a created platform window
> the QWindow geometry is not applied on the QWidget
>
>
> Diffs
> -----
>
> src/platformtheme/kdeplatformfiledialogbase.cpp b823bc7
> src/platformtheme/kdeplatformfiledialogbase_p.h 8ef5b1e
> src/platformtheme/kdeplatformfiledialoghelper.h 406a4f1
> src/platformtheme/kdeplatformfiledialoghelper.cpp 520b6f5
>
> Diff: https://git.reviewboard.kde.org/r/119594/diff/
>
>
> Testing
> -------
>
> See
> https://git.reviewboard.kde.org/r/119512/
>
>
> Thanks,
>
> Thomas Lübking
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140804/9daf560b/attachment.htm>
More information about the kde-core-devel
mailing list