Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop chant audio when chant is changed #768

Merged
merged 2 commits into from
Sep 7, 2023

Conversation

dchiller
Copy link
Collaborator

Closes #767. Playing audio is stopped when the chant is changed by:

a. Closing the open chant accordion panel;
b. Opening a new chant accordion panel; or
c. Scrolling to a new image.

@dchiller
Copy link
Collaborator Author

I'm going to revise this PR after discussion with @annamorphism ... we want to stop the sound when you open a new chant or close the current chant, but because some chants continue across folios (note open issue #628), we want users to be able to scroll to a new folio but have the sound continue.

Listens to a click on any chant accordion header to
stop audio, but keeps audio playing even if user has
scrolled to a new image.
@dchiller
Copy link
Collaborator Author

dchiller commented Sep 7, 2023

@jacobdgm @lucasmarchd01 Just putting this on your radar ... Lucas approved this but then I made substantial changes based on feedback from Anna, so would love another look from either one of you. TY!!

Copy link

@jacobdgm jacobdgm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know Backbone, but none of this looks unreasonable. The justification for this last change makes sense.

@dchiller dchiller merged commit 7de6988 into DDMAL:main Sep 7, 2023
2 checks passed
@dchiller dchiller deleted the i-767-stop-audio branch September 7, 2023 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Audio from two chants can be played at the same time
3 participants