Persist Peek volume setting across sessions

This commit implements volume persistence for the Peek utility, so that
when users adjust the volume slider for audio/video files, the volume
level is remembered and applied to subsequent files.

Changes:
- Add MediaVolume property to PeekProperties (DoubleProperty, default 1.0)
- Add MediaVolume to IUserSettings interface with getter/setter
- Implement MediaVolume in UserSettings with save-to-settings logic
- Add MediaVolume binding from MainWindow to FilePreview to AudioControl
- Add VolumeChanged events in AudioControl and FilePreview
- Track user volume changes and bubble up to MainWindowViewModel
- Handle volume changes for both audio (AudioControl) and video (MediaPlayerElement)

Fixes #31810

Author: ThanhNguyxn

src/modules/peek/Peek.FilePreviewer/Controls/AudioControl.xaml.cs

@@ -2,6 +2,7 @@
 // The Microsoft Corporation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+using System;
 using Microsoft.UI.Xaml;
 using Microsoft.UI.Xaml.Controls;
 using Peek.FilePreviewer.Previewers.MediaPreviewer.Models;
@@ -10,6 +11,10 @@ namespace Peek.FilePreviewer.Controls
 {
     public sealed partial class AudioControl : UserControl
     {
+        public event EventHandler<double>? VolumeChanged;
+
+        private bool _isSettingVolume;
+
         public static readonly DependencyProperty SourceProperty = DependencyProperty.Register(
             nameof(Source),
             typeof(AudioPreviewData),
@@ -22,6 +27,12 @@ public sealed partial class AudioControl : UserControl
             typeof(AudioControl),
             new PropertyMetadata(null));
 
+        public static readonly DependencyProperty MediaVolumeProperty = DependencyProperty.Register(
+            nameof(MediaVolume),
+            typeof(double),
+            typeof(AudioControl),
+            new PropertyMetadata(1.0, new PropertyChangedCallback((d, e) => ((AudioControl)d).MediaVolumePropertyChanged())));
+
         public AudioPreviewData? Source
         {
             get { return (AudioPreviewData)GetValue(SourceProperty); }
@@ -34,9 +45,27 @@ public string ToolTipText
             set { SetValue(ToolTipTextProperty, value); }
         }
 
+        public double MediaVolume
+        {
+            get { return (double)GetValue(MediaVolumeProperty); }
+            set { SetValue(MediaVolumeProperty, value); }
+        }
+
         public AudioControl()
         {
             this.InitializeComponent();
+            PlayerElement.MediaPlayer.VolumeChanged += MediaPlayer_VolumeChanged;
+        }
+
+        private void MediaPlayer_VolumeChanged(Windows.Media.Playback.MediaPlayer sender, object args)
+        {
+            if (!_isSettingVolume)
+            {
+                DispatcherQueue.TryEnqueue(() =>
+                {
+                    VolumeChanged?.Invoke(this, sender.Volume);
+                });
+            }
         }
 
         private void SourcePropertyChanged()
@@ -46,6 +75,32 @@ private void SourcePropertyChanged()
                 PlayerElement.MediaPlayer.Pause();
                 PlayerElement.MediaPlayer.Source = null;
             }
+            else
+            {
+                // Apply saved volume when source changes
+                ApplyVolume();
+            }
+        }
+
+        private void MediaVolumePropertyChanged()
+        {
+            ApplyVolume();
+        }
+
+        private void ApplyVolume()
+        {
+            if (PlayerElement.MediaPlayer != null)
+            {
+                _isSettingVolume = true;
+                try
+                {
+                    PlayerElement.MediaPlayer.Volume = MediaVolume;
+                }
+                finally
+                {
+                    _isSettingVolume = false;
+                }
+            }
         }
 
         private void KeyboardAccelerator_Space_Invoked(Microsoft.UI.Xaml.Input.KeyboardAccelerator sender, Microsoft.UI.Xaml.Input.KeyboardAcceleratorInvokedEventArgs args)

src/modules/peek/Peek.FilePreviewer/FilePreview.xaml

@@ -77,6 +77,7 @@
 
         <controls:AudioControl
             x:Name="AudioPreview"
+            MediaVolume="{x:Bind MediaVolume, Mode=OneWay}"
             Source="{x:Bind AudioPreviewer.Preview, Mode=OneWay}"
             ToolTipText="{x:Bind InfoTooltip, Mode=OneWay}"
             Visibility="{x:Bind IsPreviewVisible(AudioPreviewer, Previewer.State), Mode=OneWay}" />

src/modules/peek/Peek.FilePreviewer/FilePreview.xaml.cs

@@ -34,6 +34,8 @@ public sealed partial class FilePreview : UserControl, IDisposable
 
         public event EventHandler<PreviewSizeChangedArgs>? PreviewSizeChanged;
 
+        public event EventHandler<double>? MediaVolumeChanged;
+
         public static readonly DependencyProperty ItemProperty =
         DependencyProperty.Register(
             nameof(Item),
@@ -48,6 +50,13 @@ public sealed partial class FilePreview : UserControl, IDisposable
                 typeof(FilePreview),
                 new PropertyMetadata(false, async (d, e) => await ((FilePreview)d).OnScalingFactorPropertyChanged()));
 
+        public static readonly DependencyProperty MediaVolumeProperty =
+            DependencyProperty.Register(
+                nameof(MediaVolume),
+                typeof(double),
+                typeof(FilePreview),
+                new PropertyMetadata(1.0, (d, e) => ((FilePreview)d).OnMediaVolumePropertyChanged()));
+
         [ObservableProperty]
         private int numberOfFiles;
 
@@ -71,16 +80,43 @@ public sealed partial class FilePreview : UserControl, IDisposable
 
         private CancellationTokenSource _cancellationTokenSource = new();
 
+        private bool _isSettingVolume;
+
         public FilePreview()
         {
             InitializeComponent();
+
+            // Subscribe to video player volume changes
+            VideoPreview.MediaPlayer.VolumeChanged += MediaPlayer_VolumeChanged;
+
+            // Subscribe to audio player volume changes
+            AudioPreview.VolumeChanged += AudioPreview_VolumeChanged;
         }
 
         public void Dispose()
         {
+            VideoPreview.MediaPlayer.VolumeChanged -= MediaPlayer_VolumeChanged;
+            AudioPreview.VolumeChanged -= AudioPreview_VolumeChanged;
             _cancellationTokenSource.Dispose();
         }
 
+        private void MediaPlayer_VolumeChanged(Windows.Media.Playback.MediaPlayer sender, object args)
+        {
+            // Only notify if the change was made by user (not by us setting the volume)
+            if (!_isSettingVolume)
+            {
+                DispatcherQueue.TryEnqueue(() =>
+                {
+                    MediaVolumeChanged?.Invoke(this, sender.Volume);
+                });
+            }
+        }
+
+        private void AudioPreview_VolumeChanged(object? sender, double volume)
+        {
+            MediaVolumeChanged?.Invoke(this, volume);
+        }
+
         private async void Previewer_PropertyChanged(object? sender, System.ComponentModel.PropertyChangedEventArgs e)
         {
             // Fallback on DefaultPreviewer if we fail to load the correct Preview
@@ -139,6 +175,12 @@ public double ScalingFactor
             }
         }
 
+        public double MediaVolume
+        {
+            get => (double)GetValue(MediaVolumeProperty);
+            set => SetValue(MediaVolumeProperty, value);
+        }
+
         public bool MatchPreviewState(PreviewState? value, PreviewState stateToMatch)
         {
             return value == stateToMatch;
@@ -218,6 +260,30 @@ private async Task OnScalingFactorPropertyChanged()
             await UpdatePreviewSizeAsync(_cancellationTokenSource.Token);
         }
 
+        private void OnMediaVolumePropertyChanged()
+        {
+            ApplyMediaVolume();
+        }
+
+        private void ApplyMediaVolume()
+        {
+            var volume = MediaVolume;
+            _isSettingVolume = true;
+            try
+            {
+                if (VideoPreview.MediaPlayer != null)
+                {
+                    VideoPreview.MediaPlayer.Volume = volume;
+                }
+
+                AudioPreview.MediaVolume = volume;
+            }
+            finally
+            {
+                _isSettingVolume = false;
+            }
+        }
+
         private async Task UpdatePreviewSizeAsync(CancellationToken cancellationToken)
         {
             if (Previewer != null)

src/modules/peek/Peek.UI/MainWindowViewModel.cs

@@ -110,6 +110,21 @@ public int DisplayItemCount
         [ObservableProperty]
         private bool _isErrorVisible = false;
 
+        private IUserSettings UserSettings { get; }
+
+        public double MediaVolume
+        {
+            get => UserSettings.MediaVolume;
+            set
+            {
+                if (UserSettings.MediaVolume != value)
+                {
+                    UserSettings.MediaVolume = value;
+                    OnPropertyChanged();
+                }
+            }
+        }
+
         private enum NavigationDirection
         {
             Forwards,
@@ -126,9 +141,10 @@ private enum NavigationDirection
 
         private DispatcherTimer NavigationThrottleTimer { get; set; } = new();
 
-        public MainWindowViewModel(NeighboringItemsQuery query)
+        public MainWindowViewModel(NeighboringItemsQuery query, IUserSettings userSettings)
         {
             NeighboringItemsQuery = query;
+            UserSettings = userSettings;
             WindowTitle = _defaultWindowTitle;
 
             NavigationThrottleTimer.Tick += NavigationThrottleTimer_Tick;

src/modules/peek/Peek.UI/PeekXAML/MainWindow.xaml

@@ -46,8 +46,11 @@
             NumberOfFiles="{x:Bind ViewModel.DisplayItemCount, Mode=OneWay}" />
 
         <fp:FilePreview
+            x:Name="FilePreviewer"
             Grid.Row="1"
             Item="{x:Bind ViewModel.CurrentItem, Mode=OneWay}"
+            MediaVolume="{x:Bind ViewModel.MediaVolume, Mode=OneWay}"
+            MediaVolumeChanged="FilePreviewer_MediaVolumeChanged"
             NumberOfFiles="{x:Bind ViewModel.DisplayItemCount, Mode=OneWay}"
             PreviewSizeChanged="FilePreviewer_PreviewSizeChanged"
             ScalingFactor="{x:Bind ViewModel.ScalingFactor, Mode=OneWay}" />

src/modules/peek/Peek.UI/PeekXAML/MainWindow.xaml.cs

@@ -284,6 +284,11 @@ private void AppWindow_Closing(AppWindow sender, AppWindowClosingEventArgs args)
             Uninitialize();
         }
 
+        private void FilePreviewer_MediaVolumeChanged(object? sender, double volume)
+        {
+            ViewModel.MediaVolume = volume;
+        }
+
         private bool IsNewSingleSelectedItem(SelectedItem selectedItem)
         {
             try

src/modules/peek/Peek.UI/Services/IUserSettings.cs

@@ -9,5 +9,7 @@ public interface IUserSettings
         public bool CloseAfterLosingFocus { get; }
 
         public bool ConfirmFileDelete { get; set; }
+
+        public double MediaVolume { get; set; }
     }
 }

src/modules/peek/Peek.UI/Services/UserSettings.cs

@@ -38,6 +38,7 @@ private PeekSettings Settings
                     _settings = value;
                     CloseAfterLosingFocus = _settings.Properties.CloseAfterLosingFocus.Value;
                     ConfirmFileDelete = _settings.Properties.ConfirmFileDelete.Value;
+                    _mediaVolume = _settings.Properties.MediaVolume.Value;
                 }
             }
         }
@@ -49,6 +50,8 @@ private PeekSettings Settings
 
         private bool _confirmFileDelete;
 
+        private double _mediaVolume = 1.0;
+
         /// <summary>
         /// Gets or sets a value indicating whether the user is prompted before a file is recycled.
         /// </summary>
@@ -75,6 +78,29 @@ public bool ConfirmFileDelete
             }
         }
 
+        /// <summary>
+        /// Gets or sets the volume level for media playback (0.0 to 1.0).
+        /// </summary>
+        public double MediaVolume
+        {
+            get => _mediaVolume;
+            set
+            {
+                // Clamp value between 0 and 1
+                var clampedValue = Math.Max(0.0, Math.Min(1.0, value));
+                if (Math.Abs(_mediaVolume - clampedValue) > 0.001)
+                {
+                    _mediaVolume = clampedValue;
+
+                    lock (_settingsLock)
+                    {
+                        _settings.Properties.MediaVolume.Value = _mediaVolume;
+                        _settingsUtils.SaveSettings(_settings.ToJsonString(), PeekModuleName);
+                    }
+                }
+            }
+        }
+
         public UserSettings()
         {
             _settingsUtils = new SettingsUtils();

src/settings-ui/Settings.UI.Library/PeekProperties.cs

@@ -20,6 +20,7 @@ public PeekProperties()
             CloseAfterLosingFocus = new BoolProperty(false);
             ConfirmFileDelete = new BoolProperty(true);
             EnableSpaceToActivate = new BoolProperty(true); // Toggle is ON by default for new users. No impact on existing users.
+            MediaVolume = new DoubleProperty(1.0); // Default volume is 100%
         }
 
         public HotkeySettings ActivationShortcut { get; set; }
@@ -32,6 +33,8 @@ public PeekProperties()
 
         public BoolProperty EnableSpaceToActivate { get; set; }
 
+        public DoubleProperty MediaVolume { get; set; }
+
         public override string ToString() => JsonSerializer.Serialize(this, SettingsSerializationContext.Default.PeekProperties);
     }
 }