Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Android] Border with RoundRectangle - fix #21250

Merged
merged 11 commits into from
Jun 7, 2024
Merged

Conversation

kubaflo
Copy link
Collaborator

@kubaflo kubaflo commented Mar 17, 2024

Issues Fixed

Fixes #21173
Fixes #19877

Before After

@kubaflo kubaflo requested a review from a team as a code owner March 17, 2024 01:15
@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Mar 17, 2024
@PureWeen PureWeen requested review from jsuarezruiz and removed request for StephaneDelcroix March 17, 2024 01:42
@kubaflo
Copy link
Collaborator Author

kubaflo commented Mar 18, 2024

This also fixes this issue #19877

@dotnet-policy-service dotnet-policy-service bot added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Mar 18, 2024
@jsuarezruiz
Copy link
Contributor

Added pending test snapshots except on Windows where the test has not been run. Running it again.

@dotnet dotnet deleted a comment from azure-pipelines bot Apr 16, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot May 9, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot May 9, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot May 9, 2024
jsuarezruiz
jsuarezruiz previously approved these changes May 9, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot May 9, 2024
@jsuarezruiz jsuarezruiz self-requested a review May 9, 2024 15:26
jsuarezruiz
jsuarezruiz previously approved these changes May 9, 2024
@Eilon Eilon removed the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label May 10, 2024
# Conflicts:
#	src/Controls/tests/TestCases.Android.Tests/snapshots/android/BorderRoundRectangleWithImage.png
#	src/Controls/tests/TestCases.Android.Tests/snapshots/android/BorderWithRoundRectangleShouldRoundCornersOfContentWithinIt.png
#	src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue19877.cs
#	src/Controls/tests/TestCases.Shared.Tests/Tests/Issues/Issue21173.cs
#	src/Controls/tests/TestCases/Issues/Issue19877.xaml
#	src/Controls/tests/TestCases/Issues/Issue19877.xaml.cs
#	src/Controls/tests/TestCases/Issues/Issue21173.xaml
#	src/Controls/tests/TestCases/Issues/Issue21173.xaml.cs
@mattleibow
Copy link
Member

Thanks for this PR! Been a long while, so I rebased because we should have reviewed a bit sooner...

@dotnet dotnet deleted a comment from azure-pipelines bot Jun 6, 2024
@dotnet dotnet deleted a comment from azure-pipelines bot Jun 6, 2024
@dotnet dotnet deleted a comment from jsuarezruiz Jun 6, 2024
@mattleibow mattleibow added this to the .NET 8 SR7 milestone Jun 6, 2024
@mattleibow mattleibow merged commit d91e682 into dotnet:main Jun 7, 2024
49 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-border Border community ✨ Community Contribution
Projects
None yet
4 participants