Review Request 116517: Implement change volume on mouse wheel when top of the voulume icon. based on the "Bug 322821 - Make volume controllable using mouse scroll "

[Rukshan Chathuranga]

> On March 2, 2014, 11:16 a.m., Shantanu Tushar wrote:
> > Good thinking, and in fact this is the idea that first came to my mind. But, as you would've noticed, the experience on clicking the button isn't very natural now because you aren't really pressing the button.
> > Another (and better, imo) solution you can try is reverse the parenting order of MouseEventListener and ToolButton (i.e. MouseEventListener { ToolButton { } }). This way you will get both the interactions (click and scroll) and will still feel natural.

Oh really. But i can't see any diffent or any unnatural behavior after this modification. i'm sorry, may be i can't see that small difference. But my suggestion is until we find the most appropriate way, we can use this modification as the fixed one. 


- Rukshan


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


On March 1, 2014, 3:03 p.m., Rukshan Chathuranga wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116517/
> -----------------------------------------------------------
> 
> (Updated March 1, 2014, 3:03 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> Implement change volume on mouse wheel when top of the voulume icon.
> 
> 
> Diffs
> -----
> 
>   mediaelements/mediacontroller/MediaController.qml 968c459 
> 
> Diff: https://git.reviewboard.kde.org/r/116517/diff/
> 
> 
> Testing
> -------
> 
> tested the function and not change the default controlling due to new changes.
> 
> 
> Thanks,
> 
> Rukshan Chathuranga
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140302/43d5609a/attachment.html>