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