Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

UWP TitleView Width not updated after push/pop Modal #14779

Merged
merged 1 commit into from
Oct 25, 2021
Merged

Conversation

jfversluis
Copy link
Member

@jfversluis jfversluis commented Oct 22, 2021

[Takeover from #10849]

TitleViewManager is unregistered at CommandBar unload from the page calling PushModal. After the page gets visible again by a PopModal
the CommandBar is reloaded again but the TitleViewManager is not re-registered.

This is a fix for that problem.

Description of Change

Issues Resolved

API Changes

None

Platforms Affected

  • UWP

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

Testing Procedure

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

…ain after a PushModal and PopModal call.

TitleViewManager is unregistered at CommandBar unload from the page calling PushModal. After the page gets visible again by a PopModal
the CommandBar is reloaded again but the TitleViewManager is not re-registered.

This is a fix for that problem.
Copy link
Contributor

@YZahringer YZahringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jfversluis tested on several projects, LGTM 👍

@jfversluis jfversluis merged commit 0a77f0e into 5.0.0 Oct 25, 2021
5.0.0 SR7 (Planning) - Target Date Nov. 10th automation moved this from PR Needs Review to Done Oct 25, 2021
@jfversluis jfversluis deleted the takeover-10849 branch October 25, 2021 09:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[Bug] NavigationPage.TitleView do not resize after modal NavigationPage on UWP
3 participants