Review Request: Add autoscroll function to the lyrics applet.

Bart Cerneels bart.cerneels at kde.org
Thu Apr 21 08:38:29 CEST 2011


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



src/context/applets/lyrics/LyricsApplet.cpp
<http://git.reviewboard.kde.org/r/101155/#comment2445>

    Some trailing whitespace here, when committing please make sure it's gone.


- Bart


On April 19, 2011, 5:07 p.m., Jan Gerrit Marker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101155/
> -----------------------------------------------------------
> 
> (Updated April 19, 2011, 5:07 p.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Summary
> -------
> 
> The text of the current song is automatically scrolled according to the position in the song. If the scrollbar is pressed and moved the autoscroll is disabled till the scollbar is released and it can be disabled/reactivated manually by clicking on an action. Whether autoscrolling is enabled or disabled is saved over Amarok restarts.
> 
> 
> Diffs
> -----
> 
>   src/context/applets/lyrics/LyricsApplet.h 17c91bf 
>   src/context/applets/lyrics/LyricsApplet.cpp a9c6be3 
> 
> Diff: http://git.reviewboard.kde.org/r/101155/diff
> 
> 
> Testing
> -------
> 
> Tested and worked fine.
> 
> 
> Thanks,
> 
> Jan Gerrit
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110421/e28ee2dc/attachment-0001.htm 


More information about the Amarok-devel mailing list