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

[Bug] Adding controls to rows in a grid that have no RowDefinition crashes #1943

Closed
mattleibow opened this issue Aug 4, 2021 · 0 comments · Fixed by #1947
Closed

[Bug] Adding controls to rows in a grid that have no RowDefinition crashes #1943

mattleibow opened this issue Aug 4, 2021 · 0 comments · Fixed by #1947
Assignees
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3!

Comments

@mattleibow
Copy link
Member

Description

If I create a Grid, and then add a view to Grid.Row=1, it crashes.

Steps to Reproduce

  1. Add a view to a grid that has now row definitions
  2. Explode

Expected Behavior

No crash.

Actual Behavior

Crash.

Workaround

Add enough rows and columns to cover all the views.

@mattleibow mattleibow added this to the 6.0.100-preview.7 milestone Aug 4, 2021
@hartez hartez self-assigned this Aug 4, 2021
@Redth Redth added the area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter label Aug 4, 2021
@mattleibow mattleibow linked a pull request Aug 4, 2021 that will close this issue
@ghost ghost locked as resolved and limited conversation to collaborators Feb 19, 2022
@samhouts samhouts added the fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3! label Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter fixed-in-6.0.100-preview.7.3 Look for this fix in 6.0.100-preview.7.3!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants