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

Fix IOS/MacOS AvataView background color with image bug #1539

Merged
merged 4 commits into from
Nov 26, 2023

Conversation

TRybina132
Copy link
Member

@TRybina132 TRybina132 commented Nov 20, 2023

Description of Change

This bug occurs if we use any instance of class derived from GeometryGroup as default clip for image in AvatarView on MacOs or IOS. In AvatarView RoundRectangleGeometry was used as default clip for image. This caused background color to overlay image.
image

In this pull request I decided not to set clip object if AvatarView is used on IOS/MacOS with this approach background does not overlay image on IOS or MacOS and RoundRectangleGeometry still be used as default image clip for other platforms.

How it looks in MacOS
image

In IOS
image

@TRybina132
Copy link
Member Author

@dotnet-policy-service agree

@TRybina132 TRybina132 marked this pull request as ready for review November 24, 2023 16:19
Copy link
Collaborator

@brminnick brminnick left a comment

Choose a reason for hiding this comment

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

Thanks @TRybina132!!

@brminnick brminnick enabled auto-merge (squash) November 26, 2023 19:16
@brminnick brminnick merged commit 19e30af into CommunityToolkit:main Nov 26, 2023
7 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.

None yet

2 participants