ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.
Actually I see that other list items on other tabs already have path captions
like this, so it's not a newly-introduced issue.
INLINE COMMENTS
> recentusagemodel.cpp:365
> + const auto url = QUrl::fromUserInput(resourceAt(row));
> + KIO::highlightInFileManager({url});
> } else if (actionId == QLatin1String("forgetAll")) {
Probably worth just doing this in a single line rather than defining a variable
used only once
REPOSITORY
R120 Plasma Workspace
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D26121
To: meven, #vdg, #plasma, ngraham, ervin
Cc: davidre, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,
zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf,
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
