----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101968/#review4865 -----------------------------------------------------------
This review has been submitted with commit 9e66958635a7786d3896e3ad6a8aa8b0621f046b by Alex Fiestas to branch KDE/4.7. - Commit On July 16, 2011, 1:34 p.m., Alex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101968/ > ----------------------------------------------------------- > > (Updated July 16, 2011, 1:34 p.m.) > > > Review request for Plasma and Aaron J. Seigo. > > > Summary > ------- > > The bug can be reproduce: > 1-kill plasma > 2-Remove all screens leaving only the smaller one > 3-Launch plasma > 4-Add a new bigger screen > > The panel will have the old size but will be placed in the right place. > > The bug seems to be in the if/elseif block's that check conditions suck "The > old screen was smaller, checking...". The first conditional block is to know > if the config has to be saved, if the config has to be saved then none of the > checks will be perform since they're in else if, so the panel won't be > resized. > > > Diffs > ----- > > plasma/desktop/shell/panelview.cpp e31c06d > > Diff: http://git.reviewboard.kde.org/r/101968/diff > > > Testing > ------- > > I have been messing around for 1h including vertical panels, so far so good. > > > Thanks, > > Alex > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
