Everything works as expected. Reset button stops the sound, but you can also just toggle the buttons, I think that’s quite intuitive. Had an idea/nice-to-have (though I think we might’ve talked about it before?) – allowing to give a custom name to the reset button via a prop to the BaseWindow. Then we could call the Reset button "stop" in this example.
Reviewer's task:
[x] I didn't see no code smells
[x] The live preview works as expected -> no regressions found
Closes #12 #58
Everything works as expected. Reset button stops the sound, but you can also just toggle the buttons, I think that’s quite intuitive. Had an idea/nice-to-have (though I think we might’ve talked about it before?) – allowing to give a custom name to the reset button via a prop to the BaseWindow. Then we could call the Reset button "stop" in this example.
Reviewer's task: