> On 2009-04-02 15:56:56, Davide Bettio wrote: > > Screenshot: > > <http://reviewboard.kde.org> > > > > You should move it on top of the list. > >
And then having the Label, the ListView and the Button spanning two columns? See the new screenshot. > On 2009-04-02 15:56:56, Davide Bettio wrote: > > Screenshot: > > <http://reviewboard.kde.org> > > > > I think that you may try to add a checkbox to enable "autoswitch". > > You mean to make it more clear, that the default setting (0) does nothing? With a checkbox I'd change the default in fact. - Matthias ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/461/#review800 ----------------------------------------------------------- On 2009-04-02 17:20:21, Matthias Fuchs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/461/ > ----------------------------------------------------------- > > (Updated 2009-04-02 17:20:21) > > > Review request for Plasma and Davide Bettio. > > > Summary > ------- > > Whenever the user ticks "Use Tabs" the view changes and the available comics > are displayed in a listview, clicking at them marks them and adds them as > tabs if ok is pressed. Otherwise it is similar to the current situation. > > I'm not sure if I should add "Select All"-, "Deselect All"-Buttons. What do > you think? > > > Diffs > ----- > > /trunk/KDE/kdeplasma-addons/applets/comic/comic.h 946187 > /trunk/KDE/kdeplasma-addons/applets/comic/comic.cpp 946187 > /trunk/KDE/kdeplasma-addons/applets/comic/comicSettings.ui 946187 > /trunk/KDE/kdeplasma-addons/applets/comic/configwidget.h 946187 > /trunk/KDE/kdeplasma-addons/applets/comic/configwidget.cpp 946187 > > Diff: http://reviewboard.kde.org/r/461/diff > > > Testing > ------- > > > Screenshots > ----------- > > > http://reviewboard.kde.org/r/461/s/82/ > > http://reviewboard.kde.org/r/461/s/83/ > > http://reviewboard.kde.org/r/461/s/92/ > > > Thanks, > > Matthias > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel