Review Request: Add an hidden option to set a translucent background for DesktopView

[wearenotalone at freenet.de]

> On 2009-10-01 19:24:19, Aaron Seigo wrote:
> > anything but the desktop shell painting the wallpaper is poor design. the window manager does not have enough knowledge of what is happening in the shell to do so properly. what, exactly, is the benefit of letting compiz draw the wallpaper? if it's simply visual bling, i'm really not interested in encouraging it by adapting plasma-desktop to such a broken idea.

Thanks for your feedback!

My intention was on the one hand to enhance the user experience for power users and on the other hand to increase (respectively restore) the compatibility with other applications like compiz. The user should have the right and the freedom to decide for themselves. With this patch, the user is able to use the wallpaper plugin of Compiz, if he really wants to. Any other user does not have to deal with this option (because hidden), due to the default setting "false" this may not cause any disadvantages / speed penalties for him.

I use Compiz with multiple viewports; using the wallpaper plugin, I can easily distinguish between different viewports. In addition, KDE4 looks really great in cooperation with Compiz. To develop a proper mapping between the viewports of Compiz and the desktop / activities of plasma, I simply lack the experience. I did not want to wait until someone else does this, instead i wanted to tackle the problem by myself. My goal was to fix the regression in 4.3.1 compared to 4.2 with minimal impact on existing code. Primary goal was to change the existing design, behavior and code as little as possible. I also tried to stick as accurately as possible to the naming scheme and the coding style of the original authors.

I hope that you can understand my arguments and allow us to use Compiz and KDE 4 together again.

Yours sincerely,
WANA
p.s. The corrected patch will follow later.


- wearenotalone


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


On 2009-10-01 18:07:49, wearenotalone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/1755/
> -----------------------------------------------------------
> 
> (Updated 2009-10-01 18:07:49)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> Since revision 928340 (http://websvn.kde.org/?view=revision&revision=928340 ) its no longer possible to set a translucent background. Because of this you cannot use the wallpaper plugin of compiz fusion any longer with kde 4. For more information please see this bugreport: https://bugs.kde.org/show_bug.cgi?id=199869
> 
> This patch adds an hidden option named "translucentBackground" (because of Qt::WA_TranslucentBackground) in ~/.kde/share/config/plasma-desktoprc (example):
> 
> ...
> [PlasmaViews][34]
> DashboardContainment=0
> translucentBackground=true
> ...
> 
> The default value of translucentBackground is false, because this is the default behavior at the moment. I also created a patch against the kdebase-workspace-4.3.1 source package of Debian Squeeze, which works fine for me and can be found here: https://bugs.kde.org/show_bug.cgi?id=199869#c10
> 
> Best Regards,
> WANA
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/shell/desktopview.h 1027876 
>   /trunk/KDE/kdebase/workspace/plasma/desktop/shell/desktopview.cpp 1027876 
> 
> Diff: http://reviewboard.kde.org/r/1755/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> wearenotalone
> 
>