D26483: Add /usr/share/wallpapers as a default wallpaper slideshow location

David Redondo noreply at phabricator.kde.org
Wed Jan 8 09:41:09 GMT 2020


davidre added inline comments.

INLINE COMMENTS

> image.cpp:366
>      if (m_slidePaths.isEmpty()) {
> -        m_slidePaths << QStandardPaths::locateAll(QStandardPaths::GenericDataLocation, QStringLiteral("share/wallpapers"), QStandardPaths::LocateDirectory);
> +        m_slidePaths << defaultWallpaperPaths;
> +    } else {

I know this was there before but didn't work before as intended because of the wrong path but imo now that there's a default in kcfg if the path is empty it should stay empty and not magically go back to the default if the user removes every path.

REPOSITORY
  R120 Plasma Workspace

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

To: ngraham, #plasma, #vdg, ndavis, davidedmundson
Cc: davidre, broulik, davidedmundson, ndavis, 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200108/bb0aeb31/attachment.html>


More information about the Plasma-devel mailing list