Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[ios] fix memory leak in RadioButton #21151
[ios] fix memory leak in RadioButton #21151
Changes from 1 commit
2a7abf4
e3a38e2
045abb0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PureWeen maybe you remember if there is something this change would break? This makes
Border.StrokeShape
andBorder.Stroke
the same now.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be fine
Prior to converting this to
AddLogicalChild
we were already setting the parenthttps://github.com/dotnet/maui/blame/cd7a0e021d99be3ef8c7d83437fb66f5080cd5af/src/Controls/src/Core/Border/Border.cs#L67-L68
Which is why I converted it over to
AddLogicalChild
during that big pushStephanes changes with these changes should make this all work alright.
The only weird thing here is if the strokes are static resources this code is still weird, but it's already weird and this makes it less weird at least.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to copy the concept with parent resource propagation that's part of stroke?
maui/src/Controls/src/Core/Border/Border.cs
Lines 113 to 114 in 2a7abf4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For
Stroke
, I guess you could have aGradientBrush
that uses a{StaticResource}
for a color inside. That seems like it would be required.For
StrokeShape
, I guess you can do the exact thing for something like adouble
value on some shape, so I'll make the properties the same.