----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108995/ -----------------------------------------------------------
Review request for Amarok. Description ------- 1. Disabled Pre-amplifier in equalizer if it is not supported by phonon. 2. Fixed top and bottom labels of first slider. Earlier band name label was at the top and slider value label was at the bottom for first slider. 3. Removed an extra semicolon EqualizerDialog.h . This addresses bug 301311. https://bugs.kde.org/show_bug.cgi?id=301311 Diffs ----- src/dialogs/EqualizerDialog.ui 43b0187 src/EngineController.cpp 3577acf src/dialogs/EqualizerDialog.h fd9032b src/dialogs/EqualizerDialog.cpp 63d8209 Diff: http://git.reviewboard.kde.org/r/108995/diff/ Testing ------- All unit test cases passed. Note : I have tested it with gstreamer only. Xine phonon keep crashing on my PC. File Attachments ---------------- Equalizer snapshot http://git.reviewboard.kde.org/media/uploaded/files/2013/02/17/equalizer.png Thanks, Harsh Gupta
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel