meven abandoned this revision.
meven marked an inline comment as done.
meven added a comment.

  Moved to https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/31

INLINE COMMENTS

> meven wrote in recentdocuments.cpp:78
> It is needed for QString coming from KActivitiesStats, they can be 
> "/home/meven/doc.txt" or "file://" or "application://" or "http://";.
> So QUrl::fromEncoded or QUrl::fromLocalFile can't be used.
> AssumeLocalFile would be nice to use, but it requires to provide a working 
> directory. https://doc.qt.io/qt-5/qurl.html#fromUserInput-1

Well workingdirectory can be empty in fact.

REPOSITORY
  R120 Plasma Workspace

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

To: meven, #plasma, ivan, ngraham, broulik
Cc: alex, broulik, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, 
jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to