december ksplash doesn't have a preview?

[spacepenguin]

I just installed the december ksplash. It works but it has no preview picture. And it is only named "december", not as the other siduction ksplash themes with a leading "siduction-" - somehow breaks ranks...

[melmarker]

the siduction-$foo naming was braindead and a horror to script, and it is true, at the moment there is no preview for the theme. i will add one, if i remove the rc marks.

BTW: The siduction-$foo naming was from me too, but that make no sense any longer and lead only to longer pathes and chances to create typos. Mention /usr/share/siduction-artwork-kde-december/wallpaper/siduction-december and such things - nonsense - the leading siduction in the path should be enough

[spacepenguin]

Not convinced, sorry. I like to have the siduction stuff visible as siduction stuff and bundled in a bunch of siduction stuff, not lost somewhere in pieces between other artwork...

[melmarker]

tja - nicht wirklich mein Problem, ich werds nicht umschreiben

[spacepenguin]

Das hab ich mir schon gedacht, Alf wäre sonst nicht Alf . Aber ich darf das trotzdem unschön finden oder?   Ich finde einfach, branding muss auch sichtbar sein.

Wobei es ja nichtmal in jedem Fall um die Dateinamen geht. Die Datei braucht ja gar kein vorangehendes "siduction", da ist natürlich das übergeordnete Verzeichnis ausreichend. Aber die Bezeichnung in der Konfigurationsdatei des Splashs, Wallpapers etc... der in der Auswahlliste angezeigte Name, der sollte meiner Meinung nach "siduction" enthalten...

Ich meine den Namen, der da in den Theme-Dateien steht: /usr/share/kde4/apps/ksplash/Themes/releasename/Theme.rc & /usr/share/wallpapers/siduction-releasename/metadata.desktop etc.

Edit: Wobei ich gerade feststelle, dass der in der KSplash Theme-Liste angezeigte Name tatsächlich der Ordner-Name ist und nicht der in der Theme.rc angegebene. Wie unschön. Trotzdem finde ich sollte das siduction Artwork auch "siduction" im Namen haben - auch wenn das im KSplash-Fall nur im Beschreibungsfeld zu sehen ist.

[melmarker]

immer noch nö

[hendrikL]

@spacepinguin

do following as root

Code: [Select]

mv /usr/share/kde4/apps/ksplash/Themes/december/preview.png /usr/share/kde4/apps/ksplash/Themes/december/Preview.png
Or navigate to it and check it.

Then the preview (Preview.png) should work

[spacepenguin]

Small typo, big effect