Review Request: Supports creation of Comic Book Archives

Matthias Fuchs mat69 at gmx.net
Tue Apr 26 23:02:11 CEST 2011



> On April 26, 2011, 4:14 p.m., Aaron J. Seigo wrote:
> > 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 :/

Yeah, I think about only supporting tabs --> hiding them in fact if there is just one.
This alone would make the ui a little easier, as well as removing the option to hide the tabs.

Btw. do you know what I have to do that the plasma job tracker displays an error message for a specific job? Or would I have to display a seperate message via a KNotification::event?


- Matthias


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101235/#review2898
-----------------------------------------------------------


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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110426/8aa93b62/attachment.htm 


More information about the Plasma-devel mailing list