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

FlyoutHeader hangs ios application on startup #8146

Closed
BaldeGH opened this issue Jun 18, 2022 · 8 comments · Fixed by #8569
Closed

FlyoutHeader hangs ios application on startup #8146

BaldeGH opened this issue Jun 18, 2022 · 8 comments · Fixed by #8569
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout fixed-in-6.0.419 Look for this fix in 6.0.419! p/1 Work that is critical for the release, but we could probably ship without 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

@BaldeGH
Copy link

BaldeGH commented Jun 18, 2022

Description

Adding a Layout inside of the Shell.FlyoutHeader is causing the application to hang/crash on ios simulator and device

Repo

image

Steps to Reproduce

  1. Create a new maui program
  2. Add LayoutControl to AppShell.xaml's Shell.FlyoutHeader
  3. Build & Deploy

Version with bug

6.0.400 (current)

Last version that worked well

6.0.312

Affected platforms

iOS

Affected platform versions

iOS 15.4

Did you find any workaround?

Remove the VerticalStackLayout and only allow a single item in the flyout

Relevant log output

No response

@BaldeGH BaldeGH added s/needs-verification Indicates that this issue needs initial verification before further triage will happen t/bug Something isn't working labels Jun 18, 2022
@PureWeen PureWeen added area-controls-shell Shell Navigation, Routes, Tabs, Flyout and removed s/needs-verification Indicates that this issue needs initial verification before further triage will happen labels Jun 20, 2022
@lnaie
Copy link

lnaie commented Jun 20, 2022

I have this header in the AppShell.xaml, just before a <Shell.FlyoutFooterTemplate>:

    <Shell.FlyoutHeaderTemplate>
        <DataTemplate>
            <Grid HeightRequest="150">
                <Image 
                    Aspect="AspectFill" 
                    Source="flyout_logo.jpg" 
                    Opacity="0.9" />
            </Grid>
        </DataTemplate>
    </Shell.FlyoutHeaderTemplate>

It used to work just before the latest update for iOS. The app gets stuck on the loading screen with .net logo.

I commented it out and now the app goes past the loading screen, but -- yeah, it has some another issue, that I have yet to identify here in github. It's not getting better yet with these new builds.

@AidarNurmukhametov
Copy link

Hello, we are developing DevExpress .NET MAUI controls and experienced the same issue in the our demo application. This issue has a High priority for us and we do not have a workaround. Thank you for your help!

@kristinx0211 kristinx0211 added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed labels Jun 22, 2022
@kristinx0211
Copy link

verified repro on IOS with above project.

@rachelkang rachelkang added this to the 6.0-servicing milestone Jun 27, 2022
@svaldetero
Copy link

This happens in my app too. I see this error message in the Debug output when it happens:
[TableView] Warning once only: UITableView was told to layout its visible cells and other contents without being in the view hierarchy (the table view or one of its superviews has not been added to a window). This may cause bugs by forcing views inside the table view to load and perform layout without accurate information (e.g. table view bounds, trait collection, layout margins, safe area insets, etc), and will also cause unnecessary performance overhead due to extra layout passes. Make a symbolic breakpoint at UITableViewAlertForLayoutOutsideViewHierarchy to catch this in the debugger and see what caused this to occur, so you can avoid this action altogether if possible, or defer it until the table view has been added to a window. Table view: <_UIMoreListTableView: 0x7f78beb8f800; frame = (0 0; 0 0); clipsToBounds = YES; gestureRecognizers = <NSArray: 0x600002d2fd50>; layer = <CALayer: 0x6000029a4de0>; contentOffset: {0, 0}; contentSiz e: {0, 0}; adjustedContentInset: {0, 0, 0, 0}; dataSource: <UIMoreListController: 0x7f78fe764aa0>>

@PureWeen PureWeen linked a pull request Jul 6, 2022 that will close this issue
@PureWeen PureWeen modified the milestones: 6.0-servicing, 6.0-sr3 Jul 6, 2022
@samhouts samhouts added the p/1 Work that is critical for the release, but we could probably ship without label Jul 12, 2022
@pierre01
Copy link

I have the same issue when I set the Icon property in a ShellContent element in IOS

@svaldetero
Copy link

@PureWeen marked this as service release 3. Release notes from https://docs.microsoft.com/en-us/visualstudio/releases/2022/release-notes-preview#summary-of-whats-new-in-this-release-of-visual-studio-2022-version-173-preview-4 just says ".NET MAUI Servicing Updates". Which service release was included in that update? Was this item included? Anywhere someone can tie a VS Update number to a MAUI Milestone so we know when our issue is fixed?

@PureWeen PureWeen modified the milestones: 6.0-sr3, 6.0-servicing Jul 20, 2022
@pierre01
Copy link

I have not seen this issue with 6.0.419 anymore.

@PureWeen
Copy link
Member

@svaldetero it should be resolved with the latest public preview release

@ghost ghost locked as resolved and limited conversation to collaborators Aug 20, 2022
@samhouts samhouts added the fixed-in-6.0.419 Look for this fix in 6.0.419! label Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout fixed-in-6.0.419 Look for this fix in 6.0.419! p/1 Work that is critical for the release, but we could probably ship without 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

Successfully merging a pull request may close this issue.

10 participants