Review Request 118599: Presets chooser, make selected brush more visible

[Commit Hook]

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


This review has been submitted with commit 2fbebbda1f66120e9cdd08d95bffc5c340dc5e51 by Ivan Yossi to branch master.

- Commit Hook


On July 3, 2014, 7:33 a.m., Ivan Yossi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118599/
> -----------------------------------------------------------
> 
> (Updated July 3, 2014, 7:33 a.m.)
> 
> 
> Review request for Calligra, Dmitry Kazakov and Boudewijn Rempt.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> Preset list ease of read enhancement
> 
> Removed lines seem to do nothing and be redundant, at least over linux.
> 
> Reduced margin between presets rectangles, thick line for highlight is no longer needed, and on darker themes it makes the space line, of 4px, very heavy.
> Changed opacity and composition mode on highlight color to be easily visible with any theme highlight color selection.
> 
> 
> Diffs
> -----
> 
>   krita/plugins/extensions/dockers/imagedocker/image_strip_scene.cpp ce00a30 
>   krita/plugins/paintops/libpaintop/kis_brush_chooser.cpp 5f45c68 
>   krita/ui/widgets/kis_preset_chooser.cpp 8e80b4b 
> 
> Diff: https://git.reviewboard.kde.org/r/118599/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ivan Yossi
> 
>

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