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

[Android] Fix crash when opening a TabbedPage from a ContentPage that is referenced in Shell #9213

Closed
wants to merge 1 commit into from

Conversation

maxkoshevoi
Copy link
Contributor

@maxkoshevoi maxkoshevoi commented Aug 5, 2022

Description of Change

Starting from MAUI 6.0.408 there's an NRE here when trying to open a TabbedPage from a ContentPage that is referenced in Shell.

if (layoutContent.LayoutParameters is ViewGroup.MarginLayoutParams cl)

image

This will fix it (not sure if TabbedPage would start working, don't know how to test that).

Issues Fixed

Related to #7615

@ghost ghost added the community ✨ Community Contribution label Aug 5, 2022
@ghost
Copy link

ghost commented Aug 5, 2022

Hey there @maxkoshevoi! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

2 similar comments
@ghost

This comment was marked as duplicate.

@ghost

This comment was marked as duplicate.

@jfversluis

This comment was marked as outdated.

@jfversluis
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@maxkoshevoi
Copy link
Contributor Author

Could someone look at this one please? It's a blocker for me for migrating an app to MAUI since TabbedPage just throws NRE when opened

@PureWeen
Copy link
Member

The latest version of TabbedPageManager has similar code
https://github.com/dotnet/maui/blob/main/src/Controls/src/Core/Platform/Android/TabbedPageManager.cs#L265-L273

TabbedPage unfortunately still doesn't work with Shell but the fix in this PR has pretty much already been applied.

I can't really give a confident timeline on when the TabbedPage and Shell interactions will be resolved.

@PureWeen PureWeen closed this Feb 14, 2023
@maxkoshevoi maxkoshevoi deleted the mk/7615-fix branch February 15, 2023 02:40
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-tabbedpage TabbedPage community ✨ Community Contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants