Bug 163209 - Slideshow does not cycle if only one image in set directory
Summary: Slideshow does not cycle if only one image in set directory
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-04 15:48 UTC by Alan Braslau
Modified: 2008-08-06 21:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alan Braslau 2008-06-04 15:48:59 UTC
Version:           4.00.80 (KDE 4.0.80 >= (KDE 4.1 Beta1) (using 4.00.80 (KDE 4.0.80 >= (KDE 4.1 Beta1), Debian packages)
Compiler:          cc 

The Desktop settings - plasma workspace
Wallpaper dialog is incomplete:
- "None" should be renamed "Solid color"
- A true "None" is missing (i.e. don't touch background, allowing other programs to modify the X root bitmap)
- "Image" is missing positioning options (but I believe that this has been reported)
- "Slideshow" is missing the "Image" positioning options (must currently be set by first choosing "Image")
- "Slideshow" does not cycle if (only one directory containing) one image is selected! Might sound silly, but the directory contents could be dynamic or the image might be periodically changing, so it really shouldn't be cached in this case. My current work-around is to create two identical files. Plasma then periodically refreshes the background. Of course, a better solution in this case would be to manage the background via an external program, and would be possible if a true "None" option existed.
Comment 1 Aaron J. Seigo 2008-06-04 21:14:30 UTC
> don't touch background, allowing other programs to modify the X root bitmap

this is such a bad idea that all such applications deserve to be taken out back and shot.

> Of course, a better solution in this case would be to manage the background
> via an external program, and would be possible if a true "None" option existed.

that is precisely why we allow plugins for the background drawing.

in 4.2 we'll be breaking out wallpaper support from Containment so one can write a wallpaper-only-plugin.

so, revisting the remaining issues:

> - "None" should be renamed "Solid color" 

we're in a string freeze, but this change sounds sane. will have to wait until 4.2

> - "Slideshow" is missing the "Image" positioning options

already reported elsewhere

>  "Slideshow" does not cycle if (only one directory containing) one image is 
> selected!

this is the true bug here.
Comment 2 Alan Braslau 2008-06-04 23:10:16 UTC
> this is such a bad idea...
Why, does KDE claim exclusive rights to the X display?
I suppose that things have gotten so complicated that this could create havoc. Of course, I could always go back to using (v)twm, or (v)olwm if I wish!
However, I do concede that one will have to listen to the kde developers who must better understand the technical limitations to KDE that result from playing around with standard X tools.

> we allow plugins for background drawing...
Will I have to wait for 4.2 to see this? Will this be easy without a big investment in learning about kde programming? I hope so.
> this is such a bad idea...
Why, does KDE claim exclusive rights to the X display?
I suppose that things have gotten so complicated that this could create havoc. Of course, I could always go back to using (v)twm, or (v)olwm if I wish!
However, one will have to listen to the kde developers who must better understand the technical limitations to KDE resulting from playing around with standard X tools.

> we allow plugins for background drawing...
Will I have to wait for 4.2 to see this? Will this be easy without a big investment in learning about kde programming? I hope so.

> already reported elsewhere
Also missing setting the background color, useful if the images do not cover the entire screen. But I believe that this has also been already reported, as has the request for maximum aspect ratio and tiled maximum aspect ratio...

> this is a true bug here.
Indeed, this was the motivation of filing a report.


Thanks for reading the bug report. Keep up the good work. KDE 4 has come a long way since 4.0. So much so that I have now installed 4.1 beta 1 on my working terminals, living with the bugs and crashes in order to contribute by reporting the ones that are particularly hindering or else clear to express (and reproduce).

Comment 3 Alan Braslau 2008-06-17 12:17:07 UTC
The slideshow is recursive on subdirectories. Whereas this appears to be much requested, no option is currently presented to disable this feature.

Thank you
Comment 4 Alex Merry 2008-08-06 21:26:10 UTC
SVN commit 843234 by alexmerry:

Don't refuse to update just because only one file exists in a slideshow.  It may be modified by an external program.

BUG: 163209



 M  +3 -5      desktop.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=843234
Comment 5 Alex Merry 2008-08-06 21:27:46 UTC
SVN commit 843236 by alexmerry:

Backport fix: don't refuse to change the background if there is only one file in the slideshow, since it may be changed by an external program.

CCBUG: 163209



 M  +3 -5      desktop.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=843236