The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
feishin	Ready	Preview	Comment	Oct 12, 2025 5:21pm

This should also close #649 when implemented

This will have to be handled platform specific as enabling this outright will conflict with MPRIS on Linux at least. Don't know about Mac.

That aside there is probably a reason why HardwareMediaKeyHandling is on the disabled list as well, and this PR also removes that without explanation.

Leaving the line in the file as a comment also serves no purpose. If you just want to keep it for reference as how it used to be, that is what Git is for. If you're going to remove a line, remove it completely.

This was discussed in the Discord (/matrix bridge).
The correct mechanism would be to just use MediaSession API for Electron (when not using MPV player).

This will have to be handled platform specific as enabling this outright will conflict with MPRIS on Linux at least. Don't know about Mac.

One simple approach would be to enable it only on Windows via a process.platform check.
This would at least make the feature work on Windows without touching the Linux/Mac flows.

Alternatively, we could make this configurable for users who want navigator.mediaSession integration, while keeping it off by default on platforms where conflicts are known or unverified.

I would prefer you add a toggle in the settings to Enable/Disable this feature, as there are some (including myself) that wouldn't want this enabled.

Toggling the setting should prompt the user to restart the app (similar to some other settings) since it's configured within the electron main process.