D24932: Add button to open the folder in filelight to view more details

[Nathaniel Graham]

ngraham added a comment.


  Much better now, thanks. There's just one more thing I only notices now, sorry. Then I think it'll be good to go.

INLINE COMMENTS

> kpropertiesdialog.cpp:1452
> +{
> +    const QUrl url = properties->url();
> +

Instead of defining this as a variable that's used only once, you can just directly access the value from `properties->url()` in the `KRun::run()` function

REPOSITORY
  R241 KIO

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

To: shubham, ngraham, #frameworks
Cc: kde-frameworks-devel, #frameworks, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191227/5b44ca23/attachment.html>