D18809: Image Wallpaper Slideshow - display the list of images that will be shown

[Nathaniel Graham]

ngraham added inline comments.

INLINE COMMENTS

> davidre wrote in image.cpp:877
> Yes I also saw this method since the backgroundlist uses it but the Header said to use KRun if you want to open a folder instead of it.

Well it just depends on what you want to do. If you want to open the folder, then yeah KRun is better. But if you want to highlight it in a file manager window, then `highlightInFileManager()` is the way to go.

> WallpaperDelegate.qml:40
>  
>      hoverEnabled: true
>  

I just realized: setting this to false when in slideshow mode is a quick way to make them non-selectable.

> config.qml:278
> +                                    text: modelData
> +                                    visible: folderDelegate.hovered
> +                                    delay: 1000

This should also hide itself when either of the actions' tooltips is visible (otherwise they overlap each other)

REPOSITORY
  R120 Plasma Workspace

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

To: davidre, #vdg, ngraham
Cc: davidedmundson, ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190208/5dc7172e/attachment.html>