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

On June 24th, 2010, 5:38 p.m., Aaron Seigo wrote:

looks good; i've committed this patch as it seems you don't have an svn account? (if you would like to continue working on these kinds of patches, we can hook you up with one)
Thanks! That's right, no svn account yet. I would like to work on plasma more though, should I request one now or wait until I've done a few more patches?

- Anthony


On June 24th, 2010, 3:39 p.m., Anthony Bryant wrote:

Review request for Plasma.
By Anthony Bryant.

Updated 2010-06-24 15:39:08

Description

Aaron's patch almost fixed the problem, but there's still the issue that the view is getting created on startup even if its screen does not exist.
I've added a few checks in createWaitingDesktops() to make sure a view is only created if it's for a containment with a valid screen and desktop.

Testing

Started plasma with and without an external screen and tried adding and removing it a few times, with and without per virtual desktop views.

Diffs

  • /trunk/KDE/kdebase/workspace/plasma/desktop/shell/plasmaapp.cpp (1142286)

View Diff