----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101235/#review2898 -----------------------------------------------------------
took a quick look through it and didn't see anything that popped out as "wrong"; seems to work as well. the config UI and dialogs for this plasmoid needs some serious work though :/ - Aaron J. On April 26, 2011, 9:47 a.m., Matthias Fuchs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101235/ > ----------------------------------------------------------- > > (Updated April 26, 2011, 9:47 a.m.) > > > Review request for Plasma and Aaron J. Seigo. > > > Summary > ------- > > Adds possiblity to create a Comic Book Archive (.cbz) for a range: All, > BeginTo, EndTo, Range > > The job is displayed using the global job tracker, though atm. I have not > implemented the total number of files that have to be downloaded, since that > is more complicated. I might do that later. > > There is one thing I don't understand. How do I set an error that the job > tracker ui will also display it? The way I am setting the error apparntly did > not result in the plasma job ui tracker to display it. > > > Diffs > ----- > > applets/comic/CMakeLists.txt 8513d9c > applets/comic/comic.h 7134c72 > applets/comic/comic.cpp 88a2e31 > applets/comic/comicarchivedialog.h PRE-CREATION > applets/comic/comicarchivedialog.cpp PRE-CREATION > applets/comic/comicarchivedialog.ui PRE-CREATION > applets/comic/comicarchivejob.h PRE-CREATION > applets/comic/comicarchivejob.cpp PRE-CREATION > dataengines/comic/comicprovider.cpp 73f1791 > > Diff: http://git.reviewboard.kde.org/r/101235/diff > > > Testing > ------- > > > Thanks, > > Matthias > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
