Review Request: populate the settings of the rssnow-qml plasmoid

Marco Martin notmart at gmail.com
Thu Oct 18 12:02:49 UTC 2012


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



rssnow/plasma-applet-rssnow2.desktop
<http://git.reviewboard.kde.org/r/106582/#comment16232>

    should be called just rssnow



rssnow/plasma-applet-rssnow2.desktop
<http://git.reviewboard.kde.org/r/106582/#comment16233>

    library and plugin name should just overwrite the old rssnow


- Marco Martin


On Sept. 26, 2012, 3:23 p.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106582/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2012, 3:23 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> I have populated the settings of the rssnow-qml plasmoid
> and they are identical to the ones of the c++ one.
> 
> In order to do that I made the rssnow-qml plasmoid into a hybrid.
> 
> After submit of this review I believe that it is ready to be
> moved into the kdeplasma-addons and to remove the c++ one.
> (I would prefer to open a new review for the move)
> 
> Also I have pushed the latest changes(this diff) into the
> terietor/rssnow/settings branch in declarative-plasmoids
> 
> 
> Diffs
> -----
> 
>   rssnow/CMakeLists.txt 79b3581 
>   rssnow/feedsConfig.ui PRE-CREATION 
>   rssnow/generalConfig.ui PRE-CREATION 
>   rssnow/package/contents/ui/DropRssEntry.qml PRE-CREATION 
>   rssnow/package/contents/ui/ListItemEntry.qml 657b14b 
>   rssnow/package/contents/ui/config.ui e1e3409 
>   rssnow/package/contents/ui/main.qml a575c29 
>   rssnow/plasma-applet-rssnow2.desktop PRE-CREATION 
>   rssnow/rssnow.h PRE-CREATION 
>   rssnow/rssnow.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/106582/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20121018/f4414d11/attachment-0001.html>


More information about the Plasma-devel mailing list