Review Request 120399: Amarok: If waiting on the fadeout to pause, prevent pausing again.

[Martin Lyth]

> On Sept. 28, 2014, 9:45 a.m., Matěj Laitl wrote:
> > The change looks good (it is an improvement wrt current situation), but perhaps we can do what you envisioned on the bug, i.e. pause instantenously when pause is pressed within pausin fadeout?

I have experimented with that a bit. One problem I haven't been able to resolve is that if I make it pause before the fadeout is finished, when I press play again the fadeout hasn't reset, so for the first few moments it plays at a lower volume.

Though I should mention that I don't know anything about Phonon, so me not knowing how to do it doesn't say much about how difficult it actually is.


- Martin


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


On Sept. 28, 2014, 12:39 a.m., Martin Lyth wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120399/
> -----------------------------------------------------------
> 
> (Updated Sept. 28, 2014, 12:39 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Bugs: 339470
>     https://bugs.kde.org/show_bug.cgi?id=339470
> 
> 
> Repository: amarok
> 
> 
> Description
> -------
> 
> Don't call play or pause if we're already waiting on the timer to pause. See Bug 339470 for details on the bug, and the 1.5 lines of code are pretty self-explanatory.
> 
> 
> Diffs
> -----
> 
>   src/EngineController.cpp 19c483d 
> 
> Diff: https://git.reviewboard.kde.org/r/120399/diff/
> 
> 
> Testing
> -------
> 
> Still compiles, and pressing pause continuously no longer extends the timer infinitely.
> 
> 
> Thanks,
> 
> Martin Lyth
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20140928/31d98f09/attachment.html>