Merge pull request #5920 from gnattu/fix-safari-volume-10.9

jellyfin · Aug 17, 2024 · 90236c2 · 90236c2
2 parents 8bc9544 + 2305240
commit 90236c2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/components/apphost.js b/src/components/apphost.js
@@ -242,7 +242,7 @@ const supportedFeatures = function () {
         features.push('fullscreenchange');
     }
 
-    if (browser.tv || browser.xboxOne || browser.ps4 || browser.mobile) {
+    if (browser.tv || browser.xboxOne || browser.ps4 || browser.mobile || browser.ipad) {
         features.push('physicalvolumecontrol');
     }
 

diff --git a/src/plugins/htmlAudioPlayer/plugin.js b/src/plugins/htmlAudioPlayer/plugin.js
@@ -125,8 +125,10 @@ class HtmlAudioPlayer {
 
                 if (normalizationGain) {
                     self.gainNode.gain.value = Math.pow(10, normalizationGain / 20);
+                    self.normalizationGain = self.gainNode.gain.value;
                 } else {
                     self.gainNode.gain.value = 1;
+                    self.normalizationGain = 1;
                 }
                 console.debug('gain: ' + self.gainNode.gain.value);
             }).catch((err) => {
@@ -311,6 +313,9 @@ class HtmlAudioPlayer {
         function onVolumeChange() {
             if (!self._isFadingOut) {
                 htmlMediaHelper.saveVolume(this.volume);
+                if (browser.safari) {
+                    self.gainNode.gain.value = this.volume * self.normalizationGain;
+                }
                 Events.trigger(self, 'volumechange');
             }
         }