Review Request: Add autoscroll function to the lyrics applet.

Jan Gerrit Marker jangerrit at weiler-marker.com
Thu Apr 21 09:58:48 CEST 2011


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

(Updated April 21, 2011, 7:58 a.m.)


Review request for Amarok.


Changes
-------

Remove trailing whitespaces.


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 (updated)
-----

  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/334d944e/attachment.htm 


More information about the Amarok-devel mailing list