[PATCH] Different backgrounds per workspace

Aaron J. Seigo aseigo at kde.org
Wed Feb 27 12:36:46 CET 2008


On Wednesday 27 February 2008, Will Stephenson wrote:
> On Saturday 12 January 2008 11:29:04 Cody wrote:
> > Here you go.  You can save different background settings for each
> > different workspace.  Let me know what you think!
>
> Looks like this patch was never applied

good catch. while the configuration part of it is probably ok (though i 
haven't applied the ui file changes and looked at those specifically), this:

+    connect(KWindowSystem::self(), SIGNAL(currentDesktopChanged(int)), this, 
SLOT(reloadConfig()));

is completely broken for compositing window managers. we discussed a proper 
solution to this, and i've partially implemented it in the form of 
View::desktop() and Containment::StyleOption.

so now when Containment::paintInterface is called, the style option passed in 
is actually a Containment::StyleOption and it contains the desktop currently 
being rendered.

a per-workspace/desktop painting containment needs to respond to that when 
rendering its background, since these days we may be showing more than one 
desktop at a time.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080227/58675f1c/attachment.pgp 


More information about the Panel-devel mailing list