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

Commit Hook null at kde.org
Fri Sep 6 16:37:31 UTC 2013


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


This review has been submitted with commit c88b0fd7ebbe1b3bb0cc160cebe3f48be0af6e86 by Sven Brauch to branch master.

- Commit Hook


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/35f18298/attachment-0001.html>


More information about the Kde-imaging mailing list