Skip to content

[Android] Reset content margin on TabbedPage disappearing without removing tabs#33031

Merged
jfversluis merged 2 commits into30290-1from
copilot/sub-pr-32878-another-one
Dec 8, 2025
Merged

[Android] Reset content margin on TabbedPage disappearing without removing tabs#33031
jfversluis merged 2 commits into30290-1from
copilot/sub-pr-32878-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 7, 2025

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Description of Change

Fixes the NavigatingAwayFromTabbedPageResizesContentPage device test failure while preserving the modal navigation fix.

Root cause: The PR removed OnTabbedPageDisappearing entirely, which prevented both tab removal (desired) and bottom margin reset (needed). When navigating away from a TabbedPage with bottom tabs, the content margin remained set, causing subsequent pages to not use full screen height.

Solution: Re-enabled OnTabbedPageDisappearing to call SetContentBottomMargin(0) without calling RemoveTabs(). This ensures:

  • Tabs remain during modal navigation (original fix preserved)
  • Content gets full height when navigating away (device test passes)

Issues Fixed

Fixes CommunityToolkit/Maui#2708

Closes #30290


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…bottom margin on Disappearing

Co-authored-by: jfversluis <939291+jfversluis@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue with tabs removal during modal navigation on Android [Android] Reset content margin on TabbedPage disappearing without removing tabs Dec 7, 2025
Copilot AI requested a review from jfversluis December 7, 2025 07:14
@jfversluis jfversluis marked this pull request as ready for review December 8, 2025 09:00
@jfversluis jfversluis merged commit 5840400 into 30290-1 Dec 8, 2025
1 check passed
@jfversluis jfversluis deleted the copilot/sub-pr-32878-another-one branch December 8, 2025 09:00
@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants