Review Request 115892: Implement sound support for flashcard practice mode

[Inge Wallin]

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

(Updated Feb. 19, 2014, 8:05 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Edu and Amarvir Singh.


Repository: parley


Description
-------

This patch implements sounds support for flashcards in Parley.

I started to play around a bit and see how it was done in the written words practice mode and then I just copied relevant parts of the code and the ui file.  And behold!  It actually works.

There seems to be a bug in there that makes the sound button for the solution to sometimes appear and sometimes not, but I think that is a bug in the engine rather than in the flashcard class.  There is also a bug that makes the sound button itself stay in the "playing" state much longer than is warranted by the size of the sound file. But that is also not in the new code (it's also present in the previous practice modes that supported sound).


Diffs
-----

  TODO b264925 
  src/practice/flashcardmodewidget.cpp 9d7ee9f 
  src/practice/practice_widget_flashcard.ui c43a528 

Diff: https://git.reviewboard.kde.org/r/115892/diff/


Testing
-------

Created a test file with some sounds in it and practiced that N times.


Thanks,

Inge Wallin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140219/26ed74c9/attachment.html>