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

2018-01-11 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes.
Closed by commit R120:3b4be6568579: [Wallpaper Image Configuration] Set 
cfg_Image whenever delegate becomes current (authored by broulik).

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D9818?vs=25160&id=25161

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

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

To: broulik, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


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

2018-01-11 Thread David Edmundson
davidedmundson accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart


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

2018-01-11 Thread Kai Uwe Broulik
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