Review Request: permit the simple wallpaper name in the config file

[Marco Martin]

> On 2010-08-09 17:48:58, Aaron Seigo wrote:
> > another thought that occurs to me: i wonder if the "translate a package name to an absolute path" doesn't belong in the Desktop Scripting .. that way all wallpapers would get the benefit of it and it would be a penalty paid just once (at initial layout loading).
> 
> Marco Martin wrote:
>     probably...
>     fact is however my main use case was for plasma mobile, that doesn't have desktop scripting :/
>     so probablyboth are needed, this kind of resolution in image, and a convenience function is desktop scripting as well
>     
>     and what about a static Package *Package::fromName(Qstring)?
> 
> Aaron Seigo wrote:
>     "fact is however my main use case was for plasma mobile, that doesn't have desktop scripting"
>     
>     i suppose the question, then, is: should it.
>     
>     "this kind of resolution in image"
>     
>     as long as it can be done without much overhead. it will mean replicating this in every image using wallpaper, though.
>     
>     "what about a static Package *Package::fromName(Qstring)"
>     
>     it also needs a package type, since not all packages are installed and queried the same way. and then package structures would need a way to define how to do that lookup; most are currently based on using ksycoca but, as in the case of wallpaper images and themes, they are based on looking at the filesystem. so it's doable, but it's also PackageStructure-dependent.

>i suppose the question, then, is: should it.
possibly: but then we have again the problem of making the binding a public library or fork it

>as long as it can be done without much overhead. it will mean replicating this in every image using wallpaper
yes, not really good (atm there is just virus as officially distributed?) (weather is using images as well but aren't user configurable in this way)

>as in the case of wallpaper images and themes, they are based on looking at the filesystem
yeah, that's true :/


- Marco


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4947/#review6916
-----------------------------------------------------------


On 2010-08-09 15:48:31, Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4947/
> -----------------------------------------------------------
> 
> (Updated 2010-08-09 15:48:31)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> This patch makes possible to specify in the config file just the wallpaper name, such as "Ethias" or "Grass"
> this would be necessary to do default plasma layouts with a different wallpaper per containment for instance.
> the patch looks like a quick and dirty way to me, another could be a Package constructor that takesthe desktop file, or the associated KService as a path..
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/generic/wallpapers/image/image.cpp 1157559 
> 
> Diff: http://reviewboard.kde.org/r/4947/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Marco
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100810/619bda8d/attachment.htm