From 7648e43ce9e6a3c56e7bbd6e6ca5158466a4646c Mon Sep 17 00:00:00 2001
From: Dinesh Solanki <15937452+dineshsolanki@users.noreply.github.com>
Date: Fri, 2 Aug 2024 00:08:02 +0530
Subject: [PATCH 1/4] dummy: Add Video selector to HtmlBox control

Introduced a new Video model and integrated a ComboBox for video selection in the HtmlBox control. Updated the HtmlBox.cs to handle video selection and loaded videos dynamically.
---
 FoliCon/Models/Data/Video.cs  |  3 +++
 FoliCon/Views/HtmlBox.xaml    | 10 ++++++++-
 FoliCon/Views/HtmlBox.xaml.cs | 41 ++++++++++++++++++++++++++++++++++-
 3 files changed, 52 insertions(+), 2 deletions(-)
 create mode 100644 FoliCon/Models/Data/Video.cs

diff --git a/FoliCon/Models/Data/Video.cs b/FoliCon/Models/Data/Video.cs
new file mode 100644
index 0000000..a4790fd
--- /dev/null
+++ b/FoliCon/Models/Data/Video.cs
@@ -0,0 +1,3 @@
+namespace FoliCon.Models.Data;
+
+public record Video(string Name, string Id);
\ No newline at end of file
diff --git a/FoliCon/Views/HtmlBox.xaml b/FoliCon/Views/HtmlBox.xaml
index afc8438..9da4df5 100644
--- a/FoliCon/Views/HtmlBox.xaml
+++ b/FoliCon/Views/HtmlBox.xaml
@@ -5,5 +5,13 @@
              xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
              xmlns:wv2="clr-namespace:Microsoft.Web.WebView2.Wpf;assembly=Microsoft.Web.WebView2.Wpf"
              mc:Ignorable="d" Background="{DynamicResource RegionBrush}">
-    <wv2:WebView2 x:Name="Browser"/>
+    <Grid>
+        <Grid.RowDefinitions>
+            <RowDefinition Height="auto"/>
+            <RowDefinition Height="*"/>
+        </Grid.RowDefinitions>
+        <ComboBox Grid.Row="0" x:Name="VideoSelector" Margin="10" SelectionChanged="VideoSelector_OnSelectionChanged">
+        </ComboBox>
+        <wv2:WebView2 Grid.Row="1" x:Name="Browser"/>
+    </Grid>
 </UserControl>
diff --git a/FoliCon/Views/HtmlBox.xaml.cs b/FoliCon/Views/HtmlBox.xaml.cs
index 3271928..04e5689 100644
--- a/FoliCon/Views/HtmlBox.xaml.cs
+++ b/FoliCon/Views/HtmlBox.xaml.cs
@@ -1,4 +1,6 @@
-using HandyControl.Themes;
+using System.Collections;
+using HandyControl.Themes;
+using Video = FoliCon.Models.Data.Video;
 
 namespace FoliCon.Views;
 
@@ -21,12 +23,38 @@ public static readonly DependencyProperty HtmlTextProperty
             typeof(HtmlBox), 
             new PropertyMetadata(default(string), OnHtmlTextPropertyChanged));
 
+    public static readonly DependencyProperty VideosProperty
+        = DependencyProperty.Register(
+            nameof(Videos), 
+            typeof(ICollection<Video>), 
+            typeof(HtmlBox), 
+            new PropertyMetadata(default(ICollection<Video>), OnVideosPropertyChanged));
+
+    private static void OnVideosPropertyChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
+    {
+        if (d is not HtmlBox control)
+        {
+            return;
+        }
+
+        control.VideoSelector.ItemsSource = e.NewValue as ICollection;
+    }
+
     private bool IsVideoAvailable { get; set; }
 
     public HtmlBox()
     {
         InitializeComponent();
         _backgroundColor = ThemeManager.Current.ApplicationTheme == ApplicationTheme.Dark ? DarkGray : White;
+        Videos = new ObservableCollection<Video>()
+        {
+            new("Harry", "https://www.youtube.com/embed/l91Km49W9qI"),
+            new("Potter", "https://www.youtube.com/embed/DgeDDXcw--I"),
+        };
+        VideoSelector.ItemsSource = Videos;
+        VideoSelector.SelectedValuePath = "Id";
+        VideoSelector.DisplayMemberPath = "Name";
+        VideoSelector.SelectedIndex = 0;
     }
 
     public string HtmlText
@@ -35,6 +63,11 @@ public string HtmlText
         set => SetValue(HtmlTextProperty, value);
     }
     
+    public ICollection Videos
+    {
+        get => (ICollection)GetValue(VideosProperty);
+        set => SetValue(VideosProperty, value);
+    }
     private async Task ProcessBrowse()
     {
         if (Browser is not {IsLoaded: true})
@@ -122,4 +155,10 @@ private async Task InitializeAsync(string content)
                                                         </body>
                                                     </html>
                                         """;
+
+    private void VideoSelector_OnSelectionChanged(object sender, SelectionChangedEventArgs e)
+    {
+        HtmlText = VideoSelector.SelectedValue?.ToString() ?? string.Empty;
+        ProcessBrowse();
+    }
 }
\ No newline at end of file

From 0e61a9d145a972e5e3182e949347c9aa3331caf5 Mon Sep 17 00:00:00 2001
From: Dinesh Solanki <15937452+dineshsolanki@users.noreply.github.com>
Date: Fri, 2 Aug 2024 00:30:20 +0530
Subject: [PATCH 2/4] feature: Add support for displaying multiple videos

Introduced a new Videos property in ListItem to handle a collection of videos. Updated SearchResultViewModel to set this property and modified SearchResult and HtmlBox views to bind and display these videos.
---
 FoliCon/Models/Data/ListItem.cs             |  9 ++++++-
 FoliCon/ViewModels/SearchResultViewModel.cs | 30 +++++++++++++++++++--
 FoliCon/Views/HtmlBox.xaml                  |  2 +-
 FoliCon/Views/HtmlBox.xaml.cs               |  5 ----
 FoliCon/Views/SearchResult.xaml             |  2 +-
 5 files changed, 38 insertions(+), 10 deletions(-)

diff --git a/FoliCon/Models/Data/ListItem.cs b/FoliCon/Models/Data/ListItem.cs
index d70bbb6..0631843 100644
--- a/FoliCon/Models/Data/ListItem.cs
+++ b/FoliCon/Models/Data/ListItem.cs
@@ -12,6 +12,8 @@ public class ListItem : BindableBase
     private Uri _trailerLink;
     private string _id;
     private MediaType _mediaType = MediaType.Unknown;
+
+    private ICollection<Video> _videos;
     
     private string _initialPoster;
     private bool _isInitialSet;
@@ -79,7 +81,12 @@ public string TrailerKey
         get => _trailerKey;
         set => SetProperty(ref _trailerKey, value);
     }
-
+    
+    public ICollection<Video> Videos
+    {
+        get => _videos;
+        set => SetProperty(ref _videos, value);
+    }
     public string Id
     {
         get => _id;
diff --git a/FoliCon/ViewModels/SearchResultViewModel.cs b/FoliCon/ViewModels/SearchResultViewModel.cs
index 810471f..cc19df2 100644
--- a/FoliCon/ViewModels/SearchResultViewModel.cs
+++ b/FoliCon/ViewModels/SearchResultViewModel.cs
@@ -1,4 +1,5 @@
 using DelegateCommand = Prism.Commands.DelegateCommand;
+using Video = FoliCon.Models.Data.Video;
 
 namespace FoliCon.ViewModels;
 
@@ -434,10 +435,11 @@ private async Task GetGameTrailer(string itemId)
         {
             return;
         }
-
+        SetVideos(r);
         var key = r.First(v => v.Name == "Trailer");
         SetTrailer(key.VideoId);
     }
+    
 
     private async Task GetMovieTrailer(string itemId)
     {
@@ -447,6 +449,7 @@ private async Task GetMovieTrailer(string itemId)
 
             if (result != null)
             {
+                SetVideos(result.Results);
                 var trailer = ChooseTrailer(result.Results);
                 if (trailer != null)
                 {
@@ -472,7 +475,7 @@ private async Task GetTvTrailer(string itemId)
         {
             return;
         }
-
+        SetVideos(result.Results);
         var i = ChooseTrailer(result.Results);
     
         if (i != null)
@@ -499,6 +502,29 @@ private void SetTrailer(string trailerKey)
         Logger.Debug("Trailer for {Title} is {Trailer}", ResultListViewData.SelectedItem.Title,
             ResultListViewData.SelectedItem.Trailer);
     }
+
+    private void SetVideos(List<TMDbLib.Objects.General.Video> videos)
+    {
+        if (videos == null)
+        {
+            return;
+        }
+
+        var videoList = videos
+            .Where(item=> item.Site == "YouTube")
+            .Select(video => new Video(video.Name, $"https://www.youtube.com/embed/{video.Key}"))
+            .ToList();
+
+        ResultListViewData.SelectedItem.Videos = videoList;
+    }
+    
+    private void SetVideos(GameVideo[] videos)
+    {
+        var videoList = videos
+            .Select(video => new Video(video.Name, $"https://www.youtube.com/embed/{video.VideoId}"))
+            .ToList();
+        ResultListViewData.SelectedItem.Videos = videoList;
+    }
     private void ResetPoster()
     {
         ResultListViewData.SelectedItem.ResetInitialPoster();
diff --git a/FoliCon/Views/HtmlBox.xaml b/FoliCon/Views/HtmlBox.xaml
index 9da4df5..e649816 100644
--- a/FoliCon/Views/HtmlBox.xaml
+++ b/FoliCon/Views/HtmlBox.xaml
@@ -10,7 +10,7 @@
             <RowDefinition Height="auto"/>
             <RowDefinition Height="*"/>
         </Grid.RowDefinitions>
-        <ComboBox Grid.Row="0" x:Name="VideoSelector" Margin="10" SelectionChanged="VideoSelector_OnSelectionChanged">
+        <ComboBox Grid.Row="0" x:Name="VideoSelector" Margin="10" SelectionChanged="VideoSelector_OnSelectionChanged" SelectedIndex="0">
         </ComboBox>
         <wv2:WebView2 Grid.Row="1" x:Name="Browser"/>
     </Grid>
diff --git a/FoliCon/Views/HtmlBox.xaml.cs b/FoliCon/Views/HtmlBox.xaml.cs
index 04e5689..c4f9bff 100644
--- a/FoliCon/Views/HtmlBox.xaml.cs
+++ b/FoliCon/Views/HtmlBox.xaml.cs
@@ -46,11 +46,6 @@ public HtmlBox()
     {
         InitializeComponent();
         _backgroundColor = ThemeManager.Current.ApplicationTheme == ApplicationTheme.Dark ? DarkGray : White;
-        Videos = new ObservableCollection<Video>()
-        {
-            new("Harry", "https://www.youtube.com/embed/l91Km49W9qI"),
-            new("Potter", "https://www.youtube.com/embed/DgeDDXcw--I"),
-        };
         VideoSelector.ItemsSource = Videos;
         VideoSelector.SelectedValuePath = "Id";
         VideoSelector.DisplayMemberPath = "Name";
diff --git a/FoliCon/Views/SearchResult.xaml b/FoliCon/Views/SearchResult.xaml
index 2847943..c231dd1 100644
--- a/FoliCon/Views/SearchResult.xaml
+++ b/FoliCon/Views/SearchResult.xaml
@@ -158,7 +158,7 @@
                         </hc:EventTrigger>
                     </hc:Interaction.Triggers>-->
                 </ListBox>
-                <views:HtmlBox x:Name="WebBox" Grid.Row="4" Grid.Column="0" Grid.ColumnSpan="2"
+                <views:HtmlBox x:Name="WebBox" Grid.Row="4" Grid.Column="0" Grid.ColumnSpan="2" Videos="{Binding Path=SelectedItem.(data:ListItem.Videos), ElementName=ListViewResult}"
                                HtmlText="{Binding Path=SelectedItem.(data:ListItem.Trailer), ElementName=ListViewResult, TargetNullValue='Video not available!'}"
                                ScrollViewer.VerticalScrollBarVisibility="Hidden"
                                Visibility="{Binding HtmlText, Converter={StaticResource TextToVisibilityConverter}, RelativeSource={RelativeSource Self}}"

From 945bbacd944f8d8a964d6c947b2d6cdbcfc2ab8c Mon Sep 17 00:00:00 2001
From: Dinesh Solanki <15937452+dineshsolanki@users.noreply.github.com>
Date: Fri, 2 Aug 2024 22:39:20 +0530
Subject: [PATCH 3/4] fix: Rename and refactor Video model to MediaVideo

Renamed `Video` to `MediaVideo` and updated its properties. Adjusted references throughout the project, ensuring consistent naming and functionality improvements. Removed redundant trailer selection logic and optimized video processing in the SearchResultViewModel and HtmlBox components.
---
 FoliCon/Models/Data/ListItem.cs               |  4 +-
 FoliCon/Models/Data/MediaVideo.cs             |  3 +
 FoliCon/Models/Data/Video.cs                  |  3 -
 .../InvertedTextToVisibilityConverter.cs      |  2 +-
 .../Convertor/TextToVisibilityConverter.cs    |  2 +-
 FoliCon/ViewModels/SearchResultViewModel.cs   | 65 +++++++++----------
 FoliCon/Views/HtmlBox.xaml                    |  6 +-
 FoliCon/Views/HtmlBox.xaml.cs                 | 46 ++++++++-----
 8 files changed, 70 insertions(+), 61 deletions(-)
 create mode 100644 FoliCon/Models/Data/MediaVideo.cs
 delete mode 100644 FoliCon/Models/Data/Video.cs

diff --git a/FoliCon/Models/Data/ListItem.cs b/FoliCon/Models/Data/ListItem.cs
index 0631843..88c4623 100644
--- a/FoliCon/Models/Data/ListItem.cs
+++ b/FoliCon/Models/Data/ListItem.cs
@@ -13,7 +13,7 @@ public class ListItem : BindableBase
     private string _id;
     private MediaType _mediaType = MediaType.Unknown;
 
-    private ICollection<Video> _videos;
+    private ICollection<MediaVideo> _videos;
     
     private string _initialPoster;
     private bool _isInitialSet;
@@ -82,7 +82,7 @@ public string TrailerKey
         set => SetProperty(ref _trailerKey, value);
     }
     
-    public ICollection<Video> Videos
+    public ICollection<MediaVideo> Videos
     {
         get => _videos;
         set => SetProperty(ref _videos, value);
diff --git a/FoliCon/Models/Data/MediaVideo.cs b/FoliCon/Models/Data/MediaVideo.cs
new file mode 100644
index 0000000..6d0cd2b
--- /dev/null
+++ b/FoliCon/Models/Data/MediaVideo.cs
@@ -0,0 +1,3 @@
+namespace FoliCon.Models.Data;
+
+public record MediaVideo(string Name, string Url);
\ No newline at end of file
diff --git a/FoliCon/Models/Data/Video.cs b/FoliCon/Models/Data/Video.cs
deleted file mode 100644
index a4790fd..0000000
--- a/FoliCon/Models/Data/Video.cs
+++ /dev/null
@@ -1,3 +0,0 @@
-namespace FoliCon.Models.Data;
-
-public record Video(string Name, string Id);
\ No newline at end of file
diff --git a/FoliCon/Modules/Convertor/InvertedTextToVisibilityConverter.cs b/FoliCon/Modules/Convertor/InvertedTextToVisibilityConverter.cs
index d8d580f..9aa01b0 100644
--- a/FoliCon/Modules/Convertor/InvertedTextToVisibilityConverter.cs
+++ b/FoliCon/Modules/Convertor/InvertedTextToVisibilityConverter.cs
@@ -6,7 +6,7 @@ public object Convert(object value, Type targetType, object parameter, CultureIn
     {
         if (value is string text)
         {
-            return text == "Video not available!" ? Visibility.Visible : Visibility.Hidden;
+            return text is "" or "Video not available!" ? Visibility.Visible : Visibility.Hidden;
         }
 
         return Visibility.Hidden;
diff --git a/FoliCon/Modules/Convertor/TextToVisibilityConverter.cs b/FoliCon/Modules/Convertor/TextToVisibilityConverter.cs
index a09c52b..c9cf6e4 100644
--- a/FoliCon/Modules/Convertor/TextToVisibilityConverter.cs
+++ b/FoliCon/Modules/Convertor/TextToVisibilityConverter.cs
@@ -5,7 +5,7 @@ public object Convert(object value, Type targetType, object parameter, CultureIn
     {
         if (value is string text)
         {
-            return text == "Video not available!" ? Visibility.Hidden : Visibility.Visible;
+            return text is "" or "Video not available!" ? Visibility.Hidden : Visibility.Visible;
         }
 
         return Visibility.Hidden;
diff --git a/FoliCon/ViewModels/SearchResultViewModel.cs b/FoliCon/ViewModels/SearchResultViewModel.cs
index cc19df2..787f340 100644
--- a/FoliCon/ViewModels/SearchResultViewModel.cs
+++ b/FoliCon/ViewModels/SearchResultViewModel.cs
@@ -1,5 +1,4 @@
 using DelegateCommand = Prism.Commands.DelegateCommand;
-using Video = FoliCon.Models.Data.Video;
 
 namespace FoliCon.ViewModels;
 
@@ -7,6 +6,8 @@ namespace FoliCon.ViewModels;
 public class SearchResultViewModel : BindableBase, IDialogAware
 {
     private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
+    private const string YoutubeEmbedUrl = "https://www.youtube.com/embed/{0}";
+
     #region Variables
 
     private string _title = Lang.SearchResult;
@@ -436,8 +437,6 @@ private async Task GetGameTrailer(string itemId)
             return;
         }
         SetVideos(r);
-        var key = r.First(v => v.Name == "Trailer");
-        SetTrailer(key.VideoId);
     }
     
 
@@ -450,15 +449,6 @@ private async Task GetMovieTrailer(string itemId)
             if (result != null)
             {
                 SetVideos(result.Results);
-                var trailer = ChooseTrailer(result.Results);
-                if (trailer != null)
-                {
-                    SetTrailer(trailer.Key);
-                }
-                else
-                {
-                    Logger.Warn("No trailer found for {Title}", ResultListViewData.SelectedItem.Title);
-                }
             }
         }
         catch (Exception ex)
@@ -476,22 +466,12 @@ private async Task GetTvTrailer(string itemId)
             return;
         }
         SetVideos(result.Results);
-        var i = ChooseTrailer(result.Results);
-    
-        if (i != null)
-        {
-            SetTrailer(i.Key);
-        }
-        else
-        {
-            Logger.Warn("No trailer found for {Title}", ResultListViewData.SelectedItem.Title);
-        }
     }
 
-    private static dynamic ChooseTrailer(IReadOnlyCollection<dynamic> results)
+    private static Video ChooseTrailer(IReadOnlyCollection<Video> results)
     {
-        var trailerYouTube = results.FirstOrDefault(item => item?.Type == "Trailer" && item.Site == "YouTube");
-        return trailerYouTube != null ? trailerYouTube : results.FirstOrDefault();
+        var trailerYouTube = results.FirstOrDefault(item => item is { Type: "Trailer", Site: "YouTube" });
+        return trailerYouTube ?? results.FirstOrDefault();
     }
 
     private void SetTrailer(string trailerKey)
@@ -503,26 +483,41 @@ private void SetTrailer(string trailerKey)
             ResultListViewData.SelectedItem.Trailer);
     }
 
-    private void SetVideos(List<TMDbLib.Objects.General.Video> videos)
+    private void SetVideos(List<Video> videos)
     {
-        if (videos == null)
+        if (videos == null || videos.Count == 0)
         {
             return;
         }
 
-        var videoList = videos
-            .Where(item=> item.Site == "YouTube")
-            .Select(video => new Video(video.Name, $"https://www.youtube.com/embed/{video.Key}"))
-            .ToList();
+        var trailer = ChooseTrailer(videos);
+        var videoList = new List<MediaVideo>
+        {
+            new(trailer.Name, YoutubeEmbedUrl.Format(trailer.Key))
+        };
 
+        videoList.AddRange(
+            videos.Where(video => video != trailer)
+                .Select(video => new MediaVideo(video.Name, YoutubeEmbedUrl.Format(video.Key)))
+        );
+        
         ResultListViewData.SelectedItem.Videos = videoList;
     }
-    
+
     private void SetVideos(GameVideo[] videos)
     {
-        var videoList = videos
-            .Select(video => new Video(video.Name, $"https://www.youtube.com/embed/{video.VideoId}"))
-            .ToList();
+        var trailer = videos.First(v => v.Name == "Trailer");
+        
+        var videoList = new List<MediaVideo>
+        {
+            new(trailer.Name, YoutubeEmbedUrl.Format(trailer.VideoId))
+        };
+
+        videoList.AddRange(
+            videos.Where(video => video != trailer)
+                .Select(video => new MediaVideo(video.Name, YoutubeEmbedUrl.Format(video.VideoId)))
+        );
+
         ResultListViewData.SelectedItem.Videos = videoList;
     }
     private void ResetPoster()
diff --git a/FoliCon/Views/HtmlBox.xaml b/FoliCon/Views/HtmlBox.xaml
index e649816..b2e83ac 100644
--- a/FoliCon/Views/HtmlBox.xaml
+++ b/FoliCon/Views/HtmlBox.xaml
@@ -4,14 +4,16 @@
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
              xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
              xmlns:wv2="clr-namespace:Microsoft.Web.WebView2.Wpf;assembly=Microsoft.Web.WebView2.Wpf"
+             xmlns:hc="https://handyorg.github.io/handycontrol"
              mc:Ignorable="d" Background="{DynamicResource RegionBrush}">
     <Grid>
         <Grid.RowDefinitions>
             <RowDefinition Height="auto"/>
             <RowDefinition Height="*"/>
         </Grid.RowDefinitions>
-        <ComboBox Grid.Row="0" x:Name="VideoSelector" Margin="10" SelectionChanged="VideoSelector_OnSelectionChanged" SelectedIndex="0">
-        </ComboBox>
+        <hc:ComboBox Grid.Row="0" x:Name="VideoSelector" Margin="10" SelectionChanged="VideoSelector_OnSelectionChanged"
+                  SelectedIndex="0">
+        </hc:ComboBox>
         <wv2:WebView2 Grid.Row="1" x:Name="Browser"/>
     </Grid>
 </UserControl>
diff --git a/FoliCon/Views/HtmlBox.xaml.cs b/FoliCon/Views/HtmlBox.xaml.cs
index c4f9bff..d84f9e6 100644
--- a/FoliCon/Views/HtmlBox.xaml.cs
+++ b/FoliCon/Views/HtmlBox.xaml.cs
@@ -1,6 +1,5 @@
 using System.Collections;
 using HandyControl.Themes;
-using Video = FoliCon.Models.Data.Video;
 
 namespace FoliCon.Views;
 
@@ -26,18 +25,21 @@ public static readonly DependencyProperty HtmlTextProperty
     public static readonly DependencyProperty VideosProperty
         = DependencyProperty.Register(
             nameof(Videos), 
-            typeof(ICollection<Video>), 
+            typeof(ICollection<MediaVideo>), 
             typeof(HtmlBox), 
-            new PropertyMetadata(default(ICollection<Video>), OnVideosPropertyChanged));
+            new PropertyMetadata(default(ICollection<MediaVideo>), OnVideosPropertyChanged));
 
-    private static void OnVideosPropertyChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
+    private static async void OnVideosPropertyChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
     {
         if (d is not HtmlBox control)
         {
             return;
         }
-
-        control.VideoSelector.ItemsSource = e.NewValue as ICollection;
+        await control.Dispatcher.InvokeAsync(() =>
+        {
+            control.VideoSelector.ItemsSource = e.NewValue as ICollection<MediaVideo>;
+            control.VideoSelector.SelectedIndex = 0;
+        });
     }
 
     private bool IsVideoAvailable { get; set; }
@@ -46,10 +48,7 @@ public HtmlBox()
     {
         InitializeComponent();
         _backgroundColor = ThemeManager.Current.ApplicationTheme == ApplicationTheme.Dark ? DarkGray : White;
-        VideoSelector.ItemsSource = Videos;
-        VideoSelector.SelectedValuePath = "Id";
-        VideoSelector.DisplayMemberPath = "Name";
-        VideoSelector.SelectedIndex = 0;
+       InitializeVideoSelector();
     }
 
     public string HtmlText
@@ -58,9 +57,9 @@ public string HtmlText
         set => SetValue(HtmlTextProperty, value);
     }
     
-    public ICollection Videos
+    public ICollection<MediaVideo> Videos
     {
-        get => (ICollection)GetValue(VideosProperty);
+        get => (ICollection<MediaVideo>)GetValue(VideosProperty);
         set => SetValue(VideosProperty, value);
     }
     private async Task ProcessBrowse()
@@ -70,13 +69,18 @@ private async Task ProcessBrowse()
             return;
         }
 
-        var content = !IsVideoAvailable
-            ? $"""<html><body style="background-color: {_backgroundColor}"></body></html>"""
-            : GenerateHtmlContent();
+        var content = GenerateContentString();
 
         await InitializeAsync(content);
     }
 
+    private string GenerateContentString()
+    {
+        return !IsVideoAvailable
+            ? $"""<html><body style="background-color: {_backgroundColor}"></body></html>"""
+            : GenerateHtmlContent();
+    }
+
     private string GenerateHtmlContent()
     {
         return string.Format(HtmlTemplate, LangProvider.Culture.TwoLetterISOLanguageName, _backgroundColor, HtmlText);
@@ -151,9 +155,17 @@ private async Task InitializeAsync(string content)
                                                     </html>
                                         """;
 
-    private void VideoSelector_OnSelectionChanged(object sender, SelectionChangedEventArgs e)
+    private async void VideoSelector_OnSelectionChanged(object sender, SelectionChangedEventArgs e)
     {
         HtmlText = VideoSelector.SelectedValue?.ToString() ?? string.Empty;
-        ProcessBrowse();
+        await ProcessBrowse();
+    }
+    
+    private void InitializeVideoSelector()
+    {
+        VideoSelector.ItemsSource = Videos;
+        VideoSelector.SelectedValuePath = "Url";
+        VideoSelector.DisplayMemberPath = "Name";
+        VideoSelector.SelectedIndex = 0;
     }
 }
\ No newline at end of file

From 5ee57df211a1d5d1720046c35ffea68c3960c1af Mon Sep 17 00:00:00 2001
From: Dinesh Solanki <15937452+dineshsolanki@users.noreply.github.com>
Date: Fri, 2 Aug 2024 22:52:39 +0530
Subject: [PATCH 4/4] refactor: Remove unused method from SearchResultViewModel

Removed the SetTrailer method from SearchResultViewModel as it was not being used. Also cleaned up unnecessary using statement in HtmlBox.xaml.cs.
---
 FoliCon/ViewModels/SearchResultViewModel.cs | 9 ---------
 FoliCon/Views/HtmlBox.xaml.cs               | 3 +--
 2 files changed, 1 insertion(+), 11 deletions(-)

diff --git a/FoliCon/ViewModels/SearchResultViewModel.cs b/FoliCon/ViewModels/SearchResultViewModel.cs
index 787f340..4c2552f 100644
--- a/FoliCon/ViewModels/SearchResultViewModel.cs
+++ b/FoliCon/ViewModels/SearchResultViewModel.cs
@@ -474,15 +474,6 @@ private static Video ChooseTrailer(IReadOnlyCollection<Video> results)
         return trailerYouTube ?? results.FirstOrDefault();
     }
 
-    private void SetTrailer(string trailerKey)
-    {
-        ResultListViewData.SelectedItem.TrailerKey = trailerKey;
-        ResultListViewData.SelectedItem.Trailer =
-            new Uri($"https://www.youtube.com/embed/{trailerKey}");
-        Logger.Debug("Trailer for {Title} is {Trailer}", ResultListViewData.SelectedItem.Title,
-            ResultListViewData.SelectedItem.Trailer);
-    }
-
     private void SetVideos(List<Video> videos)
     {
         if (videos == null || videos.Count == 0)
diff --git a/FoliCon/Views/HtmlBox.xaml.cs b/FoliCon/Views/HtmlBox.xaml.cs
index d84f9e6..798287a 100644
--- a/FoliCon/Views/HtmlBox.xaml.cs
+++ b/FoliCon/Views/HtmlBox.xaml.cs
@@ -1,5 +1,4 @@
-using System.Collections;
-using HandyControl.Themes;
+using HandyControl.Themes;
 
 namespace FoliCon.Views;