Review Request: Fix for bug: https://bugs.kde.org/show_bug.cgi?id=200981 (duplicated actions when a screen is pluged).

Alex Fiestas alex at eyeos.org
Wed Jul 22 18:35:05 CEST 2009


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

(Updated 2009-07-22 16:35:05.705818)


Review request for Plasma.


Summary
-------

The patch fix/workaround this bug https://bugs.kde.org/show_bug.cgi?id=200981 .
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.


Diffs (updated)
-----

  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 

Diff: http://reviewboard.kde.org/r/1099/diff


Testing
-------

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


Thanks,

Alex



More information about the Plasma-devel mailing list