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

(Updated Feb. 19, 2013, 3:37 p.m.)


Review request for Amarok.


Changes
-------

1. Got rid of the config for the micro-options
2. Moved all slots to MainWindow.cpp
3. Set the default seek time for the short seek to 2 seconds
4. Made the seek action labels actually read the seek times from the config, 
formatted with KLocale::prettyFormatDuration
5. Removed spaces around the arguments in the SLOT macro calls

The Control+Left shortcut, however, conflicts with the previous browser 
shortcut.


Description
-------

Added 3 different seek options with customizable seek durations which can be 
set in the playback-config:
Control + L/R - Short seek
          L/R - Normal/ Medium seek
Shift   + L/R - Long seek


Diffs (updated)
-----

  src/amarokconfig.kcfg 3ebf71d 
  src/MainWindow.cpp 8f985dc 
  src/MainWindow.h 27725b3 

Diff: http://git.reviewboard.kde.org/r/108964/diff/


Testing
-------


Thanks,

Anmol Ahuja

_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to