Review Request 112033: Drop windowId property from PlasmaCore.Dialog
Martin Gräßlin
mgraesslin at kde.org
Mon Aug 12 17:34:45 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112033/
-----------------------------------------------------------
Review request for Plasma and Marco Martin.
Description
-------
Property windowId was according to the comments only provided because KWin needed it. KWin's usecase was to know the window Id of the window where a given item is rendered to. In QtQuick a QQuickItem provides a pointer to the QQuickWindow holding the item. This means this hack is no longer needed.
Diffs
-----
src/declarativeimports/core/dialog.h 7392b71
src/declarativeimports/core/dialog.cpp eaf3dce
Diff: http://git.reviewboard.kde.org/r/112033/diff/
Testing
-------
Yes, KWin's ThumnailItem is still working
Thanks,
Martin Gräßlin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130812/d3ffbafd/attachment.html>
More information about the Plasma-devel
mailing list