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