D5440: Fix losing wallpaper selection when clicking "Apply".

David Edmundson noreply at phabricator.kde.org
Thu Apr 13 22:14:17 UTC 2017


davidedmundson added inline comments.

INLINE COMMENTS

> config.qml:40
>      function saveConfig() {
> -        root.restoreIndex(imageWallpaper.wallpaperModel.count)
> +        root.restoreIndex()
>          imageWallpaper.commitDeletion();

Why even do anything?

You're selecting the currently selected item again. Which does nothing.

You can surely just delete this (and restoreIndex and the Connection)

REPOSITORY
  R120 Plasma Workspace

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

To: hein, #plasma, mart
Cc: davidedmundson, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170413/e2868aa2/attachment.html>


More information about the Plasma-devel mailing list