Skip to content

added flag for not requiring user action#98

Open
calym wants to merge 1 commit intomainfrom
Time-room-change
Open

added flag for not requiring user action#98
calym wants to merge 1 commit intomainfrom
Time-room-change

Conversation

@calym
Copy link
Copy Markdown

@calym calym commented Jun 28, 2019

Added a flag so that user interaction is not required to play audio files. (Specifically, this allows us to trigger audio files to project visualizations in the Time room of the Infestation project.) Playing audio and video without user interaction is disallowed by default in Chrome.

@brandonwkipp brandonwkipp requested a review from bryankennedy July 1, 2019 21:38
@brandonwkipp
Copy link
Copy Markdown
Contributor

FWIW, this looks good to me.

@bryankennedy
Copy link
Copy Markdown
Member

I know this is from a lifetime ago, but I'm just clearing out my review requests. Sorry I never saw this one before.

@bryankennedy
Copy link
Copy Markdown
Member

We're picking this work back up in https://smm.atlassian.net/browse/EXTCH-23 and https://smm.atlassian.net/browse/EXTCH-97

I think that this should be enabled in a distinct release from some of the general Stele upgrade work. I'll merge this in as a separate release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants