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

[iOS] Application stucks on iOS with combination of ScrollView, StackLayout, Editor and CollectionView #14955

Open
vg12345 opened this issue May 6, 2023 · 8 comments
Labels
area-controls-editor area-layout StackLayout, GridLayout, ScrollView, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter layout-scrollview platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@vg12345
Copy link

vg12345 commented May 6, 2023

Description

MAUI application stucks on iOS with combination of ScrollView, StackLayout, Editor and CollectionView.
Visual Studio version: 17.5.5
Here is the content of application main page:

<ScrollView>
    <StackLayout>
        <Editor/>
        <CollectionView/>
    </StackLayout>
</ScrollView>

Works fine on Xamarin iOS

Attached is the sample project:
MauiApp2.zip

Steps to Reproduce

  1. Run application on iOS
  2. Application stucks during loading of main page

Link to public reproduction project repository

https://github.com/dotnet/maui/files/11412465/MauiApp2.zip

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 16.4

Did you find any workaround?

No response

Relevant log output

No response

@vg12345 vg12345 added the t/bug Something isn't working label May 6, 2023
@drasticactions
Copy link
Contributor

drasticactions commented May 6, 2023

When you say "Simple Combination of Several Controls", how many involve "CollectionView". Do you have other examples? Could you make a repo rather than a zip?

My guess right now would be that it's an issue with StackView, CollectionView, and putting another control inside that. Replacing the StackView with a Grid and it works fine. I think it's doing recursion on the layout when you have it with the exact code you posted. CC @hartez

@drasticactions drasticactions added the s/needs-info Issue needs more info from the author label May 6, 2023
@ghost
Copy link

ghost commented May 6, 2023

Hi @vg12345. We have added the "s/needs-info" label to this issue, which indicates that we have an open question for you before we can take further action. This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@vg12345
Copy link
Author

vg12345 commented May 7, 2023

As I wrote, this is exact code that causes the app to stuck:

<ScrollView>
    <StackLayout>
        <Editor/>
        <CollectionView/>
    </StackLayout>
</ScrollView>

The example that reproduces the issue is attached. What additional info do you need?

And, yes, StackLayout can be replaces by Grid. Does it mean that this is not a problem anymore and should not be fixed?
In the real application the stack does not contain just two empty controls. It contains much more, so maintaining Grid instead of StackLayout is not very suitable, when there are twenty of them and there is a need to change the order.

@ghost ghost added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels May 7, 2023
@vg12345 vg12345 changed the title [iOS] Application stucks for simple combination of several controls on iOS [iOS] Application stucks on iOS with combination of ScrollView, StackLayout, Editor and CollectionView May 7, 2023
@drasticactions
Copy link
Contributor

And, yes, StackLayout can be replaces by Grid. Does it mean that this is not a problem anymore and should not be fixed? In the real application the stack does not contain just two empty controls. It contains much more, so maintaining Grid instead of StackLayout is not very suitable, when there are twenty of them and there is a need to change the order.

I never said it should not be fixed, I was just suggesting that you could change it to work around your issue. And as you originally wrote "Several Controls" I wanted to know if there were more cases, so that if it was caused in different combinations all would be fixed and not just one.

I did confirm your issue, I could see that, I believe, the layout is being constantly accessed and is causing the app to not start up. I was just verifying with you if there were other cases to look for. I apologize if I wasn't clear, I'm just trying to help.

@jsuarezruiz
Copy link
Contributor

@hartez Could be related to #14951?

@jsuarezruiz jsuarezruiz removed the s/needs-attention Issue has more information and needs another look label May 8, 2023
@jsuarezruiz jsuarezruiz added this to the Backlog milestone May 8, 2023
@ghost
Copy link

ghost commented May 8, 2023

We've added this issue to our backlog, and we will work to address it as time and resources allow. If you have any additional information or questions about this issue, please leave a comment. For additional info about issue management, please read our Triage Process.

@Eilon Eilon added the area-layout StackLayout, GridLayout, ScrollView, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter label May 9, 2023
@taublast
Copy link
Contributor

This could rather be related to #15189.

@XamlTest XamlTest added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jul 7, 2023
@XamlTest
Copy link
Collaborator

XamlTest commented Jul 7, 2023

Verified this on Visual Studio Enterprise 17.7.0 Preview 2.0. Repro on iOS 16.4, not repro on Windows 11 and Android 13.0-API33 with below Project:
14955.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-editor area-layout StackLayout, GridLayout, ScrollView, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter layout-scrollview platform/iOS 🍎 s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants