----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2192/#review3142 -----------------------------------------------------------
Ship it! pretty bad indeed, good catch :) - Marco On 2009-11-17 06:55:08, Bruno Bigras wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/2192/ > ----------------------------------------------------------- > > (Updated 2009-11-17 06:55:08) > > > Review request for Plasma. > > > Summary > ------- > > The comic plasmoid has a nice feature that allow to automaticaly switch > between different source of comic. > > I noticed that if you disable the "use tabs" config option, the timer was > still running. > > Also I think it don't make sense to run the timer if there's only one tab. > > Without this change we see the busy animation each 10 seconds (by default) > for no reason. > > > Diffs > ----- > > /trunk/KDE/kdeplasma-addons/applets/comic/comic.cpp 1045605 > > Diff: http://reviewboard.kde.org/r/2192/diff > > > Testing > ------- > > > Thanks, > > Bruno > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
