----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1124/#review1761 -----------------------------------------------------------
/trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp <http://reviewboard.kde.org/r/1124/#comment1149> these lines must not be removed; recalculateGeometry() may cause a change in the size of the applet, which in turn causes constraintsEvent to get called with SizeConstraint again which then call recalculateGeometry ...... great way to trigger an infinite loop ;) /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp <http://reviewboard.kde.org/r/1124/#comment1150> how well does this work with changing a panel from horizontal to vertical and back again? the user should not need to reconfigure the pager for it to look nice. - Aaron On 2009-07-25 01:11:34, Anthony Bryant wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/1124/ > ----------------------------------------------------------- > > (Updated 2009-07-25 01:11:34) > > > Review request for Plasma. > > > Summary > ------- > > This patch fixes a few problems with the pager that are especially apparent > in vertical panels. Specifically, the rows and columns are unnecessarily > swapped in recalculateGeometry() which causes bug 200013. > Also, due to the way the rectangle sizes were calculated it was possible for > certain areas to be clipped. This patch makes sure every item can fit into > the current space. > This also fixes a bug I only noticed when reading the code, where extra > unnecessary columns could be added if desktopCount % rows > 1 > > My main concern about this patch is that it removes a mechanism for ignoring > some constraints events, I think these events are needed in order to avoid > clipping some of the virtual desktops, but I'm not sure how to reproduce the > bug that made this filtering necessary in the first place. > > > This addresses bug 200013. > https://bugs.kde.org/show_bug.cgi?id=200013 > > > Diffs > ----- > > /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.h 1002081 > /trunk/KDE/kdebase/workspace/plasma/applets/pager/pager.cpp 1002081 > > Diff: http://reviewboard.kde.org/r/1124/diff > > > Testing > ------- > > Tested in vertical and horizontal panels and on the desktop. > > > Thanks, > > Anthony > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
