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