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

Startup sound #55

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Startup sound #55

wants to merge 1 commit into from

Conversation

stianjensen
Copy link
Member

No description provided.

sigvef
sigvef previously requested changes Jul 31, 2022
renin/src/renin.ts Outdated Show resolved Hide resolved
@sigvef
Copy link
Member

sigvef commented Jul 31, 2022

Only works on Firefox, I think 🤔

@stianjensen
Copy link
Member Author

Huh, I don't remember anymore which browser I tested it in, but will make sure to test in both Firefox and Chrome!

I was already a bit skeptical that the concept of a boot sound can even keep working, given how restrictive browsers are these days on autoplaying audio before any user interaction.

@sigvef
Copy link
Member

sigvef commented Aug 8, 2022

I'm guessing it will also only work on localhost

@iver56
Copy link
Member

iver56 commented Aug 9, 2022

I still remember the days of <HEAD><BGSOUND SRC="mysound.wav"></HEAD>

Thanks to Truck!

(Will only play if nin is not already muted on launch)
@stianjensen
Copy link
Member Author

Only works on Firefox

It did work when I tested in Chrome now 🤔

@stianjensen stianjensen marked this pull request as ready for review May 27, 2023 13:24
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.

3 participants