CommunityToolkit / WindowsCommunityToolkit

The Windows Community Toolkit is a collection of helpers, extensions, and custom controls. It simplifies and demonstrates common developer tasks building .NET apps with UWP and the Windows App SDK / WinUI 3 for Windows 10 and Windows 11. The toolkit is part of the .NET Foundation.
https://docs.microsoft.com/windows/communitytoolkit/
Other
5.88k stars 1.38k forks source link

DataGrid crash on width change #2592

Open Dondon841 opened 5 years ago

Dondon841 commented 5 years ago

I'm submitting a...

Bug report (I searched for similar issues and did not find one)

Current behavior

DataGrid has a column with Drag edge of page to make the window narrower. The App will crash with "Layout cycle detected. Layout could not complete."

I stopped it crashing. The DataGrid is inside a Grid. Set the MinWidth of the Grid to greater than the minimum width that the DataGrid can be. PS it still crashes if I drag the window to span two monitors.

Expected behavior

Adjust width to follow cursor without crashing

Minimal reproduction of the problem with instructions

Environment

Nuget Package(s): 

Package Version(s): 

Windows 10 Build Number:
- [x] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] Insider Build (build number: )

App min and target version:
- [x] Fall Creators Update (16299)
- [ ] April 2018 Update (17134)
- [ ] October 2018 Update (17763)
- [ ] Insider Build (xxxxx)

Device form factor:
- [x] Desktop
- [ ] Mobile
- [ ] Xbox
- [ ] Surface Hub
- [ ] IoT

Visual Studio 
- [x] 2017 (version: )
- [ ] 2017 Preview (version: )
nmetulev commented 5 years ago

@Dondon841, could you share a repro project?

ping @MSRegisB

michael-hawker commented 5 years ago

@nmetulev should we put bugs like this in the 5.0 milestone to track for closing the release?

Dondon841 commented 5 years ago

Project files as requested. This App is only for internal use, not public release.

MainPage.xaml.cs.txt MainPage.xaml.txt

Version 5 seems to crash less often than Version 4, but still does crash.

Dondon841 commented 5 years ago

Set line 32 to "950" and it doesn't crash, but of course the behaviour is different. Remove line 29 and it crashes when you adjust the horizontal width.

Dondon841 commented 5 years ago

Any news please?

windowstoolkitbot commented 5 years ago

This issue seems inactive. It will automatically be closed in 14 days if there is no activity.

Dondon841 commented 5 years ago

I was still hoping for a response.

michael-hawker commented 5 years ago

@RBrid think we can get this fixed in the next couple of weeks for the 5.1 release?

RBrid commented 5 years ago

Yes, I will investigate this issue in the coming days. Thanks.

windowstoolkitbot commented 5 years ago

This issue seems inactive. Do you need help to complete this issue?

Dondon841 commented 5 years ago

Looks like the developers are investigating - still an active issue.

windowstoolkitbot commented 5 years ago

This issue seems inactive. It will automatically be closed in 14 days if there is no activity.

Dondon841 commented 5 years ago

Still waiting for progress.

Kyaa-dost commented 4 years ago

@RBrid was there any update on this?

CC: @anawishnoff @harinikmsft

ghost commented 3 years ago

Thanks Dondon841 for patiently waiting while the team is still investigating the issue.

@anawishnoff @RBrid can we please have an update on this issue? Thanks

ghost commented 3 years ago

Thanks Dondon841 for patiently waiting while the team is still investigating the issue.

@anawishnoff @RBrid can we please have an update on this issue? Thanks

ghost commented 3 years ago

Thanks Dondon841 for patiently waiting while the team is still investigating the issue.

@anawishnoff @RBrid can we please have an update on this issue? Thanks

ghost commented 3 years ago

Thanks Dondon841 for patiently waiting while the team is still investigating the issue.

@anawishnoff @RBrid can we please have an update on this issue? Thanks