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
  Wait for `m_ready` until doing anything. This keeps the slideshow from first 
showing the default wallpaper, then a random slideshow image, and then fade to 
the next slideshow image.

TEST PLAN
  - Switching slideshow wallpapers is still animated
  - clicking "Next wallpaper" in context menu still works
  - Copying a file to the picture folder will add it to the slideshow just fine
  
  Removed the `setSingleImage()` call in the constructor which would never do 
anything as it's guarded by `m_ready`

REPOSITORY
  R120 Plasma Workspace

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

AFFECTED FILES
  wallpapers/image/image.cpp

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

Reply via email to