----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4676/#review6644 -----------------------------------------------------------
i love the amount of red in this one :) only comment i have is that if the close button were moved into FilteringWidget itself, and a "closeClicked()" signal added to it, then the somewhat nasty static_cast to assign filteringLayout could be removed and all of that code could be encapsulated in FilteringWidget. with that done, i think this should go into svn :) - Aaron On 2010-07-17 23:05:49, Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/4676/ > ----------------------------------------------------------- > > (Updated 2010-07-17 23:05:49) > > > Review request for Plasma. > > > Summary > ------- > > Make AbstractIconList inherit from Plasma::ScrollWidget, has discussed on > plasma-devel. > > The horizontal orientation behaved a bit strangely: AbstractIconList was > becoming much larger than the screen width. I had to change the layout code > to include the "Close" button inside FilteringWidget layout instead of > creating another layout. > > Note: you need http://reviewboard.kde.org/r/4675/ to get proper scrollbar > slider sizes. > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/libs/plasmagenericshell/abstracticonlist.h > 1147944 > trunk/KDE/kdebase/workspace/libs/plasmagenericshell/abstracticonlist.cpp > 1147944 > > trunk/KDE/kdebase/workspace/libs/plasmagenericshell/widgetsexplorer/appletslist.h > 1147944 > > trunk/KDE/kdebase/workspace/libs/plasmagenericshell/widgetsexplorer/appletslist.cpp > 1147944 > > trunk/KDE/kdebase/workspace/libs/plasmagenericshell/widgetsexplorer/widgetexplorer.cpp > 1147944 > > Diff: http://reviewboard.kde.org/r/4676/diff > > > Testing > ------- > > Tested in both horizontal and vertical modes, with lists larger and smaller > than the view. > > > Thanks, > > Aurélien > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
