Review Request 119558: Krita: Add labels for nameless sliders

Boudewijn Rempt boud at valdyas.org
Thu Aug 7 20:29:32 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119558/#review64011
-----------------------------------------------------------


This looks pretty much perfect! Except for one niggle, and I'm not sure about that -- it looks like the actual curve widget is now really small, and the labels sort of disconnect from it. But maybe it always was like that -- check http://www.valdyas.org/~boud/curvewidget.png and tell me whether I'm missing something or not. When I'm sure about that part, ship it!

- Boudewijn Rempt


On Aug. 1, 2014, 1:54 a.m., Scott Petrovic wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119558/
> -----------------------------------------------------------
> 
> (Updated Aug. 1, 2014, 1:54 a.m.)
> 
> 
> Review request for Calligra and Boudewijn Rempt.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> add labels for nameless sliders
> 
> 
> Diffs
> -----
> 
>   krita/plugins/extensions/dockers/defaultdockers/wdglayerbox.ui d52b2f8 
>   krita/plugins/paintops/libpaintop/forms/wdgautobrush.ui 2f56bc1 
>   krita/plugins/paintops/libpaintop/forms/wdgcurveoption.ui ee8d229 
>   krita/plugins/paintops/libpaintop/kis_curve_option_widget.cpp c3ea0e8 
> 
> Diff: https://git.reviewboard.kde.org/r/119558/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Scott Petrovic
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140807/c9e49c7a/attachment.htm>


More information about the calligra-devel mailing list