From 4941d663b41726c5f6235d8679be238c054ae7fc Mon Sep 17 00:00:00 2001
From: vvitalii <vitaliyvovchok@gmail.com>
Date: Wed, 18 Dec 2024 01:05:47 -0800
Subject: [PATCH] windows DisconnectHandler test

---
 Example/Business/UI/Pages/MainPage.xaml       | 41 +++++++++++--------
 Example/Business/UI/Pages/MainPage.xaml.cs    |  6 +++
 .../UI/ViewModels/MainPageViewModel.cs        |  6 +++
 Example/Example.csproj                        |  2 +
 Example/Properties/launchSettings.json        |  2 +-
 Maui.PDFView/Maui.PDFView.csproj              |  4 +-
 .../Platforms/Windows/PdfViewHandler.cs       | 15 +++++--
 7 files changed, 53 insertions(+), 23 deletions(-)

diff --git a/Example/Business/UI/Pages/MainPage.xaml b/Example/Business/UI/Pages/MainPage.xaml
index c86e493..0ffaeb7 100644
--- a/Example/Business/UI/Pages/MainPage.xaml
+++ b/Example/Business/UI/Pages/MainPage.xaml
@@ -5,27 +5,26 @@
     xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
     xmlns:pdf="clr-namespace:Maui.PDFView;assembly=Maui.PDFView"
     xmlns:toolkit="http://schemas.microsoft.com/dotnet/2022/maui/toolkit"
-    Title="Example">
+    Title="Example"
+    Unloaded="ContentPage_Unloaded">
 
     <ContentPage.Behaviors>
-        <toolkit:EventToCommandBehavior
-            Command="{Binding AppearingCommand}"
-            EventName="Appearing" />
+        <toolkit:EventToCommandBehavior Command="{Binding AppearingCommand}" EventName="Appearing" />
     </ContentPage.Behaviors>
 
     <Grid RowDefinitions="Auto,*">
 
         <Grid
-            RowDefinitions="Auto,Auto"
-            ColumnDefinitions="Auto,Auto,Auto"
-            ColumnSpacing="8">
+            ColumnDefinitions="Auto,Auto,Auto,Auto"
+            ColumnSpacing="8"
+            RowDefinitions="Auto,Auto">
 
             <Label
                 Grid.Row="0"
                 Grid.Column="0"
-                Text="Horizontal"
                 FontSize="Caption"
                 HorizontalTextAlignment="Center"
+                Text="Horizontal"
                 VerticalOptions="Center" />
 
             <CheckBox
@@ -40,33 +39,41 @@
                 Grid.Column="1"
                 FontSize="Caption"
                 HorizontalTextAlignment="Center"
-                Text="{Binding MaxZoom, StringFormat='Max zoom is {0}'}">
-            </Label>
+                Text="{Binding MaxZoom, StringFormat='Max zoom is {0}'}" />
 
             <Stepper
                 Grid.Row="1"
                 Grid.Column="1"
-                Minimum="1"
                 Maximum="5"
-                Value="{Binding MaxZoom}"/>
+                Minimum="1"
+                Value="{Binding MaxZoom}" />
 
 
             <Button
                 Grid.Row="0"
-                Grid.Column="2"
                 Grid.RowSpan="2"
-                VerticalOptions="Center"
+                Grid.Column="2"
                 Command="{Binding ChangeUriCommand}"
-                Text="ChangeUri" />
+                Text="ChangeUri"
+                VerticalOptions="Center" />
+
+            <Button
+                Grid.Row="0"
+                Grid.RowSpan="2"
+                Grid.Column="3"
+                Command="{Binding OpenNewPageCommand}"
+                Text="Open new page"
+                VerticalOptions="Center" />
 
         </Grid>
 
         <pdf:PdfView
+            x:Name="pdfView"
             Grid.Row="1"
             IsHorizontal="{Binding IsHorizontal}"
-            Uri="{Binding PdfSource}"
             MaxZoom="{Binding MaxZoom}"
-            PageChangedCommand="{Binding PageChangedCommand}"/>
+            PageChangedCommand="{Binding PageChangedCommand}"
+            Uri="{Binding PdfSource}" />
     </Grid>
 
 </ContentPage>
\ No newline at end of file
diff --git a/Example/Business/UI/Pages/MainPage.xaml.cs b/Example/Business/UI/Pages/MainPage.xaml.cs
index a86a904..fbd0024 100644
--- a/Example/Business/UI/Pages/MainPage.xaml.cs
+++ b/Example/Business/UI/Pages/MainPage.xaml.cs
@@ -8,4 +8,10 @@ public MainPage()
         InitializeComponent();
         BindingContext = new MainPageViewModel();
     }
+
+    private void ContentPage_Unloaded(object sender, EventArgs e)
+    {
+        pdfView.Handler?.DisconnectHandler();
+    }
+
 }
\ No newline at end of file
diff --git a/Example/Business/UI/ViewModels/MainPageViewModel.cs b/Example/Business/UI/ViewModels/MainPageViewModel.cs
index 716e8fd..ebaf57e 100644
--- a/Example/Business/UI/ViewModels/MainPageViewModel.cs
+++ b/Example/Business/UI/ViewModels/MainPageViewModel.cs
@@ -1,6 +1,7 @@
 using CommunityToolkit.Mvvm.ComponentModel;
 using CommunityToolkit.Mvvm.Input;
 using Example.Business.Services;
+using Example.Business.UI.Pages;
 using Maui.PDFView.Events;
 using System.Diagnostics;
 using System.Windows.Input;
@@ -29,5 +30,10 @@ [RelayCommand] private void PageChanged(PageChangedEventArgs args)
         {
             Debug.WriteLine($"Current page: {args.CurrentPage} of {args.TotalPages}");
         }
+
+        [RelayCommand] private void OpenNewPage()
+        {
+            App.Current.MainPage.Navigation.PushAsync(new MainPage());
+        }
     }
 }
diff --git a/Example/Example.csproj b/Example/Example.csproj
index f303e3f..c028400 100644
--- a/Example/Example.csproj
+++ b/Example/Example.csproj
@@ -32,6 +32,8 @@
 		<ApplicationDisplayVersion>1.0</ApplicationDisplayVersion>
 		<ApplicationVersion>1</ApplicationVersion>
 
+		<WindowsPackageType Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'windows'">None</WindowsPackageType>
+
 		<SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'ios'">11.0</SupportedOSPlatformVersion>
 		<SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'maccatalyst'">13.1</SupportedOSPlatformVersion>
 		<SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'android'">21.0</SupportedOSPlatformVersion>
diff --git a/Example/Properties/launchSettings.json b/Example/Properties/launchSettings.json
index edf8aad..4f85793 100644
--- a/Example/Properties/launchSettings.json
+++ b/Example/Properties/launchSettings.json
@@ -1,7 +1,7 @@
 {
   "profiles": {
     "Windows Machine": {
-      "commandName": "MsixPackage",
+      "commandName": "Project",
       "nativeDebugging": false
     }
   }
diff --git a/Maui.PDFView/Maui.PDFView.csproj b/Maui.PDFView/Maui.PDFView.csproj
index f039b72..0ae0149 100644
--- a/Maui.PDFView/Maui.PDFView.csproj
+++ b/Maui.PDFView/Maui.PDFView.csproj
@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
 	<PropertyGroup>
 		<TargetFrameworks>net8.0;net8.0-android;net8.0-ios;net8.0-maccatalyst</TargetFrameworks>
@@ -21,7 +21,7 @@
 		<Description>MAUI library for displaying PDF files</Description>
 		<PackageProjectUrl>https://github.com/vitalii-vov/Maui.PDFView</PackageProjectUrl>
 		<PackageLicenseExpression>MIT</PackageLicenseExpression>
-		<Version>1.0.6</Version>
+		<Version>1.0.7-alpha</Version>
 		<PackageReadmeFile>README.md</PackageReadmeFile>
 	</PropertyGroup>
 
diff --git a/Maui.PDFView/Platforms/Windows/PdfViewHandler.cs b/Maui.PDFView/Platforms/Windows/PdfViewHandler.cs
index cd76492..e127c4d 100644
--- a/Maui.PDFView/Platforms/Windows/PdfViewHandler.cs
+++ b/Maui.PDFView/Platforms/Windows/PdfViewHandler.cs
@@ -57,15 +57,24 @@ protected override ScrollViewer CreatePlatformView()
                 VerticalScrollBarVisibility = Microsoft.UI.Xaml.Controls.ScrollBarVisibility.Visible,
             };
 
-            // Attach scroll event to track page changes
-            _scrollViewer.ViewChanged += OnScrollViewerViewChanged;
-
             _stack = new StackPanel { Orientation = Orientation.Vertical };
             _scrollViewer.Content = _stack;
 
             return _scrollViewer;
         }
 
+        protected override void ConnectHandler(ScrollViewer platformView)
+        {
+            _scrollViewer.ViewChanged += OnScrollViewerViewChanged;
+            base.ConnectHandler(platformView);
+        }
+
+        protected override void DisconnectHandler(ScrollViewer platformView)
+        {
+            _scrollViewer.ViewChanged -= OnScrollViewerViewChanged;
+            base.DisconnectHandler(platformView);
+        }
+
         async Task RenderPages()
         {
             _stack.Children.Clear();