Closed Bug 1192978 Opened 9 years ago Closed 7 years ago

Debounce tutorial button clicks, only load videos once

Categories

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::First Time Experience
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: sfoster, Unassigned, Mentored)

References

Details

(Whiteboard: [good first bug]) 

To avoid doing redundant work and getting into bad state, we should handle rapid clicks on the FTU tutorial buttons. The tutorial should only be started once and videos should be loaded only once
See bug 1186572 for the discussion that prompted this bug. The FTU is not ideal for a first bug just because its a certified app and requires a flashable device. But if you can do that, this is very approachable. We just need to ensure we don't do the same work repeatedly when starting and progressing through the tutorial. The Tutorial.next and Tutorial.back methods could be debounced (tutorial.js), and the letsGoButton and updateLetsGoButton in UIManager (ui.js) need to only respond to the first click. We should unit test the latter.
Mentor: sfoster
Whiteboard: [good first bug]
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.