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

Fix safari volume being reset when track changed #5923

Merged

Conversation

gnattu
Copy link
Member

@gnattu gnattu commented Aug 17, 2024

Changes

This is a follow-up of #5920. That PR missed the case that the gain value will be reset during track change. This PR always set the gain value relative to current volume on Safari

Issues

@gnattu gnattu requested a review from a team as a code owner August 17, 2024 17:57
@thornbill thornbill added this to the v10.9.10 milestone Aug 17, 2024
@thornbill thornbill added bug Something isn't working regression We broke something stable backport Backport into the next stable release labels Aug 17, 2024
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

ESLint doesn't pass. Please fix all ESLint issues.

src/plugins/htmlAudioPlayer/plugin.js Outdated Show resolved Hide resolved
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

@thornbill thornbill merged commit 6a8f21e into jellyfin:release-10.9.z Aug 18, 2024
11 checks passed
joshuaboniface pushed a commit that referenced this pull request Aug 25, 2024
Fix safari volume being reset when track changed

Original-merge: 6a8f21e

Merged-by: thornbill <thornbill@users.noreply.github.com>

Backported-by: Joshua M. Boniface <joshua@boniface.me>
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression We broke something
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants