[Kde-imaging] Review Request 112431: Display KIPImageList load/save buttons by default

Gilles Caulier caulier.gilles at gmail.com
Fri Sep 6 16:22:05 UTC 2013


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

Ship it!


Ship It!

- Gilles Caulier


On Sept. 1, 2013, 6:14 p.m., Sven Brauch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112431/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2013, 6:14 p.m.)
> 
> 
> Review request for Kipi-plugins.
> 
> 
> Description
> -------
> 
> KPImagesList has a nice feature to load or save a list of images, which is very useful for e.g. the slideshow plugin. However the buttons are not displayed by default, since the flags were apparently forgotten to be added to the default constructor. This patch fixes that.
> 
> 
> Diffs
> -----
> 
>   common/libkipiplugins/widgets/kpimageslist.cpp 78d3ee9 
> 
> Diff: http://git.reviewboard.kde.org/r/112431/diff/
> 
> 
> Testing
> -------
> 
> Load/Save buttons are displayed in the slideshow plugin, which does not explicitly specify which buttons it wants.
> 
> 
> Thanks,
> 
> Sven Brauch
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-imaging/attachments/20130906/e599c4c1/attachment.html>


More information about the Kde-imaging mailing list