Review Request: Fix for bug: (duplicated actions when a screen is pluged).

Alex Fiestas alex at
Wed Jul 22 18:00:30 CEST 2009

This is an automatically generated e-mail. To reply, visit:

Review request for Plasma.


The patch fix/workaround this bug .
I've added a new slot called existingContainmentAdded in plasmapp.cpp, is almost like containmentAdded slot but the new one doesn't add the actions.
Then, I've had to create a new signal in corona (kdelibs) desired to be emmited when an already existing containment is added.
And finally, I replaced containmentAdded signal by the new one in desktopcorona.cpp.

I feel the need of add a new slot because containmentAdded (slot) is used in more than one place, so add an argument is not an option.
Also, I took a look at containment class and containmentPrivate, but I haven't found a good place to add anything that could fix this situation.


  trunk/KDE/kdelibs/kdecore/kconfig_compiler/kconfig_compiler.cpp 1000737 
  trunk/KDE/kdebase/workspace/plasma/shells/desktop/desktopcorona.cpp 1000737 
  trunk/KDE/kdebase/workspace/plasma/shells/desktop/plasmaapp.h 1000737 
  trunk/KDE/kdebase/workspace/plasma/shells/desktop/plasmaapp.cpp 1000737 
  trunk/KDE/kdelibs/plasma/corona.h 1000737 



I test this patch using 1 and 2 monitors with a few different configurations, everything works well and the actions are never duplicated.



More information about the Plasma-devel mailing list