D9818: [Wallpaper Image Configuration] Set cfg_Image whenever delegate becomes current

Kai Uwe Broulik noreply at phabricator.kde.org
Thu Jan 11 14:29:51 UTC 2018


broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  The delete button changes the currentIndex to something else when deleting the current image but that didn't update cfg_Image. Just listen to when we become current. Might also fix keyboard navigation (didn't test if that worked before but it certainly does now).
  
  BUG: 388805

TEST PLAN
  Clicked another image, it got selected, Apply applied it.
  Clicked an image, deleted it, another image got selected, Apply was enabled and applied it.
  Verified that the Apply button is disabled when opening the dialog

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  wallpapers/image/imagepackage/contents/ui/WallpaperDelegate.qml

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


More information about the Plasma-devel mailing list