D12986: [Trash KCM] Clean up and standardize UI to be in line with the KDE HIG

Nathaniel Graham noreply at phabricator.kde.org
Thu May 24 13:22:26 UTC 2018


ngraham added inline comments.

INLINE COMMENTS

> davidedmundson wrote in kcmtrash.cpp:303
> There's no benefit in changing the initial parent of these.
> 
> It's all moot as QFormLayout::addItem takes ownership

Ah right, I think I wrote this before I found that out. In fact, you can omit the parent entirely, so I think I'll do that.

REPOSITORY
  R241 KIO

BRANCH
  clean-up-trash-kcm (branched from master)

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

To: ngraham, #dolphin, #frameworks, #vdg, davidedmundson
Cc: davidedmundson, abetts, rkflx, kfm-devel, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180524/629ceb10/attachment.html>


More information about the Kde-frameworks-devel mailing list