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

"mute" works incorrectly #46

Closed
howdiditcometothis opened this issue Apr 13, 2021 · 5 comments
Closed

"mute" works incorrectly #46

howdiditcometothis opened this issue Apr 13, 2021 · 5 comments

Comments

@howdiditcometothis
Copy link

On Vivaldi and Chrome, when I click the "vol" to mute the sound, it mutes the sound but the gui doesn't change unless I click again. On this second click, it unmutes but the gui changes to the "muted" state.

Also sometimes it seems to stop playback after finishing a song instead of looping or changing songs.

@mon
Copy link
Owner

mon commented Apr 13, 2021

That's concerning, I hope chromium hasn't changed anything to do with their audio model. Does the console provide any hints in error messages? This one's a bit tricky to debug from here.

@howdiditcometothis
Copy link
Author

Checking console I've got one "red" error: "Failed to load resource: net::ERR_BLOCKED_BY_CLIENT HwU4Afx3ZsFv3HCMpwtQftaMMBI.js:1" as the first console entry

and one yellow warning: "DevTools failed to load SourceMap: Could not load content for https://0x40.mon.im/css/hues-min.css.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE"

@mon
Copy link
Owner

mon commented Apr 14, 2021

Both of those are fine, one is cloudflare's analytics being blocked, one is a sourcemap I didn't upload.

Is it specific songs? Repeatable? If you then manually change song does it change?

@howdiditcometothis
Copy link
Author

howdiditcometothis commented Apr 21, 2021

Sorry for the delay. I've done a little testing:
Autosong is on and set to 2 loops for both checks.

test performed on a site hosted on my own locally:
Played the kanye song normally, switched to a different song with autosong, clicked mute, audio muted but ui didn't change, moved slider and audio unmutes (ui is still in unmuted state), let it play 2 loops and autosong changes song, audio mutes and ui goes to mute ui with () and at bottom of bar, clicking mute unmutes audio but ui still shows muted, autosong changes song and ui changes to "unmuted" state (both audio and ui are "unmuted"), dragging slider to 0 works normally and doesn't unmute after autosong changes.

On your site:
Clicking mute will mute audio but ui is unchanged. Moving slider will unmute audio, ui is still unmuted. Changing songs manually or with auto song, the next song will be muted on play, with unmuted ui. (edit) another click of the button changes the ui state to mute and audio is still muted.

Doesn't seem to depend on the song or whether it's autosong or manual

@mon
Copy link
Owner

mon commented Jul 8, 2021

I believe this is now fixed in v3.9, feel free to re-open if I missed an edge case.

@mon mon closed this as completed Jul 8, 2021
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

No branches or pull requests

2 participants