Review Request: Make sure vertical slider's handle have the same shadow as the horizontal one

[Commit Hook]

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


This review has been submitted with commit e975ad45bcd7058bfa9e94a63d15c56c5a06a30b by Martin Klapetek to branch master.

- Commit Hook


On June 16, 2012, 12:03 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105269/
> -----------------------------------------------------------
> 
> (Updated June 16, 2012, 12:03 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> The vertical sliders are rotated horizontal sliders including all the light effects, which looks bad as it appears like there are two different light sources. This patch rotates back the slider handle, so all the handles have consistent shadowing. The grooves should also have consistent shadow, but that may come in later patch (also it's way less visible).
> 
> See screenshots.
> 
> 
> Diffs
> -----
> 
>   plasma/declarativeimports/plasmacomponents/qml/Slider.qml 5867164 
> 
> Diff: http://git.reviewboard.kde.org/r/105269/diff/
> 
> 
> Testing
> -------
> 
> Tested on widget gallery.
> 
> 
> Screenshots
> -----------
> 
> Before
>   http://git.reviewboard.kde.org/r/105269/s/603/
> After
>   http://git.reviewboard.kde.org/r/105269/s/604/
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120616/a7eac464/attachment-0001.html>