-----------------------------------------------------------
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
> 
>

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to