UI test for #1986 - #2088
Conversation
|
Thanks for your contribution! The build and test process is starting. This may take a while. All packages have been packed successfully! 📦✅ The packages will be available for 30 days, you can either use them directly, or wait for this PR to be merged to have them published to NuGet.org. Tests will start now, you can monitor their progress below or at the actions tab. Test Results Summary (Failure) ❌ 😥
0 skipped. |
There was a problem hiding this comment.
Pull request overview
Adds an Avalonia-specific UI regression test intended to cover LiveCharts2 issue #1986 (ScrollViewer + TabControl causing content not to render after tab switching), along with a new Avalonia sample view used by the test.
Changes:
- Add a new
SharedUITeststest (AvaloniaTests) targeting the reported TabControl/ScrollViewer rendering regression. - Add a new Avalonia sample view (
VisualTest/VirtualizationTest) containing a TabControl with ScrollViewers and charts to reproduce the scenario.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| tests/SharedUITests/AvaloniaTests.cs | Adds a new Avalonia-only UI test meant to validate charts render after tab switching + scrolling. |
| samples/AvaloniaSample/VisualTest/VirtualizationTest/View.axaml.cs | Adds code-behind helpers (tab switching, scrolling) and currently auto-drives the view on Loaded. |
| samples/AvaloniaSample/VisualTest/VirtualizationTest/View.axaml | Adds the UI layout (TabControl with two ScrollViewer pages containing a large spacer + chart). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@beto-rodriguez I've opened a new pull request, #2089, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
Thanks for your contribution! The build and test process is starting. This may take a while. Test Results Summary (Failure) ❌ 😥
0 skipped. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Thanks for your contribution! The build and test process is starting. This may take a while. All packages have been packed successfully! 📦✅ The packages will be available for 30 days, you can either use them directly, or wait for this PR to be merged to have them published to NuGet.org. Tests will start now, you can monitor their progress below or at the actions tab. Test Results Summary (Failure) ❌ 😥
0 skipped. Test Results Summary (Failure) ❌ 😥
0 skipped. |
|
Thanks for your contribution! The build and test process is starting. This may take a while. All packages have been packed successfully! 📦✅ The packages will be available for 30 days, you can either use them directly, or wait for this PR to be merged to have them published to NuGet.org. Tests will start now, you can monitor their progress below or at the actions tab. Test Results Summary (Failure) ❌ 😥
0 skipped. |
|
Thanks for your contribution! The build and test process is starting. This may take a while. Test Results Summary (Failure) ❌ 😥
0 skipped. |
|
Thanks for your contribution! The build and test process is starting. This may take a while. All packages have been packed successfully! 📦✅ The packages will be available for 30 days, you can either use them directly, or wait for this PR to be merged to have them published to NuGet.org. Tests will start now, you can monitor their progress below or at the actions tab. Test Results Summary (Failure) ❌ 😥
0 skipped. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thanks for your contribution! The build and test process is starting. This may take a while. All packages have been packed successfully! 📦✅ The packages will be available for 30 days, you can either use them directly, or wait for this PR to be merged to have them published to NuGet.org. Tests will start now, you can monitor their progress below or at the actions tab. Test Results Summary (Passed) ✅ 🥳0 skipped. |
Add a UI test for #1986