Skip to content

Conversation

@beto-rodriguez
Copy link
Collaborator

some hot fixes to rc-6

Domik234 and others added 9 commits September 12, 2025 14:43
…or Android

Add a null check for _customScaleListener in DisposeController in src/_Shared.Native/Platforms/Android/PointerController.cs to prevent crashes when navigating away without touching the chart. Only dispose _customScaleListener if it is not null.
…stener

Fixes #1971 - Android Fix: Add a null check for _customScaleListener in DisposeController to prevent crash
Update Windows App SDK version to fix AOT issues
Generators works better from nuget (at least on Mac), and lets reference the source code in this repo instead of NuGet packages.
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request contains hotfixes for the 2.0.0-rc6.1 release, primarily focusing on dependency updates and sample refactoring.

  • Updated Microsoft.WindowsAppSDK package versions to 1.7.250606001 across WinUI projects
  • Fixed a potential null reference issue in Android pointer controller disposal
  • Refactored chart sections samples to use data binding instead of hardcoded values

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
LiveChartsCore.SkiaSharpView.WinUI.csproj Updated WindowsAppSDK and SDK.BuildTools package versions
PointerController.cs Added null-conditional operator for safe disposal
WinUISample.csproj Updated WindowsAppSDK package version
Various View.xaml files Changed hardcoded section coordinates to data binding
ViewModel.cs files Added properties for section coordinates or removed unused imports
template.md Fixed broken link reference
Directory.Build.props Updated version and build configuration flags

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@beto-rodriguez beto-rodriguez merged commit b82ca03 into master Sep 13, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants