Page MenuHomePhabricator

Skip back in recitation (was: Re-listening to recitation)
Closed, ResolvedPublic1 Estimated Story Points

Description

Recitation can be rewinded to allow re-listening
License: MIT/GPL
Existing: Partially, https://github.com/westonruter/html5-audio-read-along
To do: Partially existing, (click on the first word or drag the playback indicator backwards). Add a button and/or a keyboard shortcut. Define which functions are needed, in addition to play/pause.

Identified as a component during the pilot study.

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
Jopparn renamed this task from [Task] Re-listening to [Task] Re-listening to recitation (Wikispeech).Apr 28 2016, 6:52 AM

After a quick test of this functionality, I can't see the benefit of dragging the playback indicator, as opposed to clicking where you want to go in the text.

When the user skips back, the player either skips to the beginning of the current interval or to the beginning of the previous interval. The latter will happen if the current position in the interval is "early enough". This means that a user skipping back twice, does not get stuck restarting the same interval over and over.

"Early enough" can be implemented in two ways:

  1. Time distance (to start of interval) is less than x seconds. This means that the first skip can sometimes take you to the previous interval.
  2. Two skip commands are issued within s seconds. The first skip always takes you to the start of the interval.

This needs to be implemented in addition to T133688.

I tested how some media players handled skipping tracks, which is similar behaviour. I found two things happening when pressing the "previous" button:

  1. Starting playing the previous track. (VLC, Podcast Addict)
  2. Starting playing the previuos track if the current time was < ~3 s, otherwise restarted the current track. (Spotify, Rhythmbox)
Lokal_Profil renamed this task from [Task] Re-listening to recitation (Wikispeech) to Skip back in recitation (was: Re-listening to recitation).Oct 4 2016, 12:20 PM
Lokal_Profil changed the point value for this task from 2 to 1.5.

Decision is to implement this as follows:

  • Skipping sentences/utterances: If more than X seconds has been listened to skipping takes you back to start of that utterance, otherwise it takes you to the previous one. The number of seconds can be set in the config.
  • Skipping in words/tokens: Always skips to previous token.

Worked on in Wikispeech (Sprint 2016-09-21):

  • Rough implementation of skipping in utterances

To do in Wikispeech (Sprint 2016-10-05):

  • Implementation of skipping in tokens
  • Clean up and review of skipping in utterances

Change 314664 had a related patch set uploaded (by Sebastian Berlin (WMSE)):
Skip back sentence

https://gerrit.wikimedia.org/r/314664

Change 315218 had a related patch set uploaded (by Sebastian Berlin (WMSE)):
Skip back word

https://gerrit.wikimedia.org/r/315218

Sebastian_Berlin-WMSE changed the point value for this task from 1.5 to 0.5.

Worked on in Wikispeech (Sprint 2016-10-05):

  • Finished skipping utterances.
  • Implemented skipping tokens.
  • Committed for review.

To do in Wikispeech (Sprint 2016-10-19):

  • Review.
Sebastian_Berlin-WMSE changed the point value for this task from 0.5 to 1.5.

Worked on in Wikispeech (Sprint 2016-11-02):

  • First review.

To do in Wikispeech (Sprint 2016-11-16):

  • Implementation based on first review.

Worked on in Wikispeech (Sprint 2016-11-16):

  • Second patch reviewed
  • Awaiting patch for T140105 being merged

To do in Wikispeech (Sprint 2016-11-30):

  • Implement reviews
Lokal_Profil changed the point value for this task from 1.5 to 1.

Worked on in Wikispeech (Sprint 2016-11-30):

  • Review implemented for first patch
  • Review partially implemented for second patch

To do in Wikispeech (Sprint 2016-12-14):

  • Re-review
  • Merge for fame and glory

Change 314664 merged by jenkins-bot:
Skip back sentence

https://gerrit.wikimedia.org/r/314664