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

VisualElement.OnMeasure currently routes to some inconsistent logic compared to OnMeasureOverride #23244

Open
PureWeen opened this issue Jun 25, 2024 · 0 comments
Labels
area-controls-general General issues that span multiple controls, or common base classes such as View or Element s/triaged Issue has been reviewed
Milestone

Comments

@PureWeen
Copy link
Member

PureWeen commented Jun 25, 2024

Description

VE.OnMeasure follows this path

image

Which calls this

image

It should probably call this in both places of the OnMeasure override

image

The checks on the constraints of "0" also is going to cause inconsistent behavior. MAUI treats zero as zero.

We should review all the measure code inside VisualElement and just force everything to route to the MAUI paths.

Right now we hit a lot of inconsistent behavior because

button.Measure
and
(button as IView).Measure

Do different things.

we should make it so those both do the same thing and then if users need to switch back to old behavior they can via a switch.

@dotnet-policy-service dotnet-policy-service bot added the s/triaged Issue has been reviewed label Jun 25, 2024
@PureWeen PureWeen added this to the 9.0-preview7 milestone Jun 25, 2024
@PureWeen PureWeen added the area-controls-general General issues that span multiple controls, or common base classes such as View or Element label Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-general General issues that span multiple controls, or common base classes such as View or Element s/triaged Issue has been reviewed
Projects
Status: Todo
Development

No branches or pull requests

1 participant