Skip to content

Fix audio muting and slider not working, save state stutter, and mobile context menu appearing on desktops. - #1264

Open
joethun wants to merge 1 commit into
EmulatorJS:mainfrom
joethun:main
Open

joethun wants to merge 1 commit into
EmulatorJS:mainfrom
joethun:main

Conversation

@joethun

@joethun joethun commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

This fixes issues #1191 and #1074 and also an issue where the audio controls don't work.

@joethun joethun changed the title Fix audio muting and slider not working, save state stutter, and context menu appearing on desktops. Fix audio muting and slider not working, save state stutter, and mobile context menu appearing on desktops. Sep 6, 2026

@ethanaobrien ethanaobrien left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Would you be able to add support to use openal if available?

Comment thread data/src/GameManager.js Outdated
}
getState() {
return this.Module.EmulatorJSGetState();
const info = this.Module.UTF8ToString(this.functions.saveStateInfo()).split("|");

@ethanaobrien ethanaobrien Sep 20, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Isn't this the same code that EmulatorJSGetState is lol - if there's a speed issue here that I'm probably missing probably a better idea to upstream it to the RetroArch script then here

https://github.com/EmulatorJS/RetroArch/blob/170ffa717026f4cc1c17a6096d09a245e4f9b448/emscripten/emulatorjs.js#L3

Comment thread data/src/emulator.js
this.startGameError(this.localization("Failed to start game"));
});
}
withCoreAudioCapture(callback) {

@ethanaobrien ethanaobrien Sep 20, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did you attempt building RetroArch with set_volume exposed btw? Iirc this is how we had volume control before OpenAL. I'm not sure how I feel about proxying the audio context

@ethanaobrien

Copy link
Copy Markdown
Member

Sorry about conflicts

@joethun

joethun commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

RetroArch repo pull request here

This branch has not been deployed

No deployments
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.

2 participants