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 app poorly responsive when many elements, nested deeply, are on the screen #12592

Closed
BioTurboNick opened this issue Jan 12, 2023 · 9 comments
Labels
legacy-area-perf Startup / Runtime performance platform/iOS 🍎 t/bug Something isn't working t/perf The issue affects performance (runtime speed, memory usage, startup time, etc.) (sub: perf)
Milestone

Comments

@BioTurboNick
Copy link
Contributor

BioTurboNick commented Jan 12, 2023

Description

The same page was very responsive in Xamarin.Forms 5, and in MAUI 7 on Android.

But on MAUI 7 iOS, the touch interaction responses tend to be delayed and executed in bursts.

The problem doesn't happen if I only have one custom control on the screen, but does happen when I have 12 of them.

It also only happens with several nested levels of controls.

Steps to Reproduce

Several nested levels of Grid objects, the innermost one containing complex controls composed of many layout containers.

Link to public reproduction project repository

https://github.com/BioTurboNick/iOSSlowdownExample

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

iOS 15

Did you find any workaround?

No response

Relevant log output

No response

@BioTurboNick BioTurboNick added the t/bug Something isn't working label Jan 12, 2023
@jsuarezruiz jsuarezruiz added platform/iOS 🍎 legacy-area-perf Startup / Runtime performance labels Jan 12, 2023
@rachelkang rachelkang added the s/needs-repro Attach a solution or code which reproduces the issue label Jan 12, 2023
@ghost
Copy link

ghost commented Jan 12, 2023

Hi @BioTurboNick. We have added the "s/needs-repro" label to this issue, which indicates that we require steps and sample code to reproduce the issue before we can take further action. Please try to create a minimal sample project/solution or code samples which reproduce the issue, ideally as a GitHub repo that we can clone. See more details about creating repros here: https://github.com/dotnet/maui/blob/main/.github/repro.md

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.

@BioTurboNick BioTurboNick changed the title iOS app poorly responsive when many elements are on the screen iOS app poorly responsive when many elements, nested deeply, are on the screen Jan 14, 2023
@BioTurboNick
Copy link
Contributor Author

@rachelkang Reproduction project added.

@ghost ghost added s/needs-attention Issue has more information and needs another look and removed s/needs-repro Attach a solution or code which reproduces the issue labels Jan 14, 2023
@BioTurboNick
Copy link
Contributor Author

Possibly related: rotating the device with this page leads to a visibly slow layout cycle.

@hartez
Copy link
Contributor

hartez commented Jan 17, 2023

#12627 will likely help with this.

@BioTurboNick
Copy link
Contributor Author

That would be great!

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

ghost commented Jan 30, 2023

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@BioTurboNick
Copy link
Contributor Author

I believe this has been fixed by 7.0.59. I updated the workloads and the delays are vastly reduced, if not gone. There's still a visibly slow rotation-related layout cycle. But maybe that's another issue?

@hartez
Copy link
Contributor

hartez commented Feb 8, 2023

I believe this has been fixed by 7.0.59. I updated the workloads and the delays are vastly reduced, if not gone. There's still a visibly slow rotation-related layout cycle. But maybe that's another issue?

Oh, rotation is probably a whole other ball of wax. If you've got a repro project that shows really rough performance on rotation, please open an issue for that!

@hartez
Copy link
Contributor

hartez commented Mar 3, 2023

Closing this, since the OP believes it's been fixed. @BioTurboNick if you're still running into issues with rotation, please open an issue and we'll take a look.

@hartez hartez closed this as completed Mar 3, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Apr 2, 2023
@Eilon Eilon added the t/perf The issue affects performance (runtime speed, memory usage, startup time, etc.) (sub: perf) label May 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
legacy-area-perf Startup / Runtime performance platform/iOS 🍎 t/bug Something isn't working t/perf The issue affects performance (runtime speed, memory usage, startup time, etc.) (sub: perf)
Projects
None yet
Development

No branches or pull requests

5 participants