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

Implemented Geometry.GetWidenedGeometry. #12724

Merged
merged 6 commits into from
Aug 31, 2023
Merged

Conversation

grokys
Copy link
Member

@grokys grokys commented Aug 30, 2023

What does the pull request do?

Implements Geometry.GetWidenedGeometry. Similar to the WPF method Geometry.GetWidenedPathGeometry except that it doesn't return a PathGeometry and there is no overload with tolerance settings.

@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0038940-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

grokys added 2 commits August 30, 2023 17:57
Otherwise we'd be changing the behavior of an existing API (`StrokeContains`) which should be discussed first.
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0038954-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

- Move the logic into `SKPathHelper.CreateStrokedPath`; called from `PathHelper` and `GetWidenedGeometry`
- Use a pen hash code to check if we're up-to-date in `PathHelper` and include the dash style in that
@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0039027-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@avaloniaui-team
Copy link
Contributor

You can test this PR using the following package version. 11.0.999-cibuild0039034-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@grokys grokys marked this pull request as ready for review August 31, 2023 12:44
@grokys grokys added this pull request to the merge queue Aug 31, 2023
Merged via the queue into master with commit c8da11f Aug 31, 2023
@grokys grokys deleted the feature/widened-geometry branch August 31, 2023 13:25
@grokys grokys added backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch and removed backport-candidate-11.0.x Consider this PR for backporting to 11.0 branch labels Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants