Review Request: changes to comic applet and dataengine and an osnews provider

[Marco Martin]

> On 2008-02-03 10:17:39, Anne-Marie Mahfouf wrote:
> > Works well especially the resizing which was broken. Note that XKCD is still broken but it was so before your patch.
> > How does OSNEWS focus work? Is it the RSS feed that changes the picture? Or is it done at some time intervalle?

the rss feed shows the latest issues, and then i must parse the article page pinted in the rss feed link.
the downside of using rss is that only last 15-20 comics will be available, but it's almost impossible to do it in another way because they are managing them like normal news items, so they don't yet have the usual prev/next links present in other webcomics


- Marco


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://matt.rogers.name/r/47/#review35
-----------------------------------------------------------


On 2008-02-03 09:45:54, Marco Martin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://matt.rogers.name/r/47/
> -----------------------------------------------------------
> 
> (Updated 2008-02-03 09:45:54)
> 
> 
> Review request for Plasma.
> 
> 
> Summary
> -------
> 
> this is a quite massive change to the comic applet, yeah i know quite hard to review sorry for that.
> the main chanea are:
> -i've added a provider to http://www.osnews.com/comic (got permission from them) it uses a rss feed so only last 15 will be available but i think is enough
> -now the applet can be resized (option that can be disabled if you want to see the comic as is)
> -the dataengine now giver more data that can also be cached at the moment it has only the url of the comic of the day, but i plan to use it also to pass the date of the previous and the next comic (useful for comics that don't have a strip every day)
> -the url of the comic is displayed at the bottom of the applet and clicking on it it opens a web browser on that page (useful if the comic is scaled down and some (osnews) explicitly asks it)
> 
> 
> Diffs
> -----
> 
>   /trunk/extragear/plasma/applets/comic/CMakeLists.txt
>   /trunk/extragear/plasma/applets/comic/comic.h
>   /trunk/extragear/plasma/applets/comic/comic.cpp
>   /trunk/extragear/plasma/applets/comic/configdialog.h
>   /trunk/extragear/plasma/applets/comic/configdialog.cpp
>   /trunk/extragear/plasma/applets/comic/configimg.qrc
>   /trunk/extragear/plasma/applets/comic/pics/osnews.png
>   /trunk/extragear/plasma/dataengines/comic/CMakeLists.txt
>   /trunk/extragear/plasma/dataengines/comic/cachedprovider.h
>   /trunk/extragear/plasma/dataengines/comic/cachedprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/comic.h
>   /trunk/extragear/plasma/dataengines/comic/comic.cpp
>   /trunk/extragear/plasma/dataengines/comic/comicprovider.h
>   /trunk/extragear/plasma/dataengines/comic/comicprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/dilbertprovider.h
>   /trunk/extragear/plasma/dataengines/comic/dilbertprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/garfieldprovider.h
>   /trunk/extragear/plasma/dataengines/comic/garfieldprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/osnewsprovider.h
>   /trunk/extragear/plasma/dataengines/comic/osnewsprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/snoopyprovider.h
>   /trunk/extragear/plasma/dataengines/comic/snoopyprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/userfriendlyprovider.h
>   /trunk/extragear/plasma/dataengines/comic/userfriendlyprovider.cpp
>   /trunk/extragear/plasma/dataengines/comic/xkcdprovider.h
>   /trunk/extragear/plasma/dataengines/comic/xkcdprovider.cpp
> 
> Diff: http://matt.rogers.name/r/47/diff
> 
> 
> Testing
> -------
> 
> -tried the osnews provider with every comic they've published for now
> -scaling down a comic if not all the strips have the same size it keeps the right aspect ratio for every comic
> -tried the link with all the comics
> -xkcd comic seems broken, was it broken already (i'll do some tests)
> 
> 
> Screenshots
> -----------
> 
> osnews favicon
>   http://matt.rogers.name/r/47/s/2/
> 
> 
> Thanks,
> 
> Marco
> 
>