D22738: Remove some uses on QDesktopWidget

Aleix Pol Gonzalez noreply at phabricator.kde.org
Fri Jul 26 01:11:17 BST 2019


apol marked an inline comment as done.
apol added inline comments.

INLINE COMMENTS

> davidedmundson wrote in foldermodel.cpp:1640
> KNewFileMenu.cpp in the constructor
>  1148     d->m_parentWidget = qobject_cast<QWidget *>(parent);
> 
> without a transient parent we probably have focus issues?

It doesn't show any issues that I can see.
Also the QDesktopWidget isn't an actual widget so I'm not sure what it's supposed to be bringing.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D22738

To: apol, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190726/3d6b0592/attachment.html>


More information about the Plasma-devel mailing list