D26448: Add KRecentFilesMenu to replace KRecentFileAction
Kai Uwe Broulik
noreply at phabricator.kde.org
Tue Jan 7 09:32:02 GMT 2020
broulik added inline comments.
INLINE COMMENTS
> krecentfilesmenu.cpp:150
> + // Don't restore if file doesn't exist anymore
> + if (url.isLocalFile() && !QFile::exists(url.toLocalFile())) {
> + continue;
If we're rewriting this thing anyway, can we please make sure it does not block application startup checking if those files exist, as can happen if you had opened files on an NFS mount before.
Either do it only when the menu is opened (aboutToShow) and/or asynchronously.
REPOSITORY
R236 KWidgetsAddons
REVISION DETAIL
https://phabricator.kde.org/D26448
To: nicolasfella, #frameworks, dfaure
Cc: broulik, elvisangelaccio, cfeck, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200107/fa60ef17/attachment.html>
More information about the Kde-frameworks-devel
mailing list