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] Fix ShellContent Title Does Not Update at Runtime #26062

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

devanathan-vaithiyanathan
Copy link
Contributor

@devanathan-vaithiyanathan devanathan-vaithiyanathan commented Nov 22, 2024

Root cause

The issue occurred because the ShellContent title did not update dynamically in response to property changes. On iOS, real-time updates were not reflected as the Title property changes were not being observed.

Description of Change

Implemented dynamic title updates for ShellContent by subscribing to the PropertyChanged event. This ensures that any changes to the Title property are observed and updated in real-time, resolving the issue where the title did not reflect updates dynamically.

Regarding Test case

I did not add a new test case because the test cases added in PR for Windows and Android. These tests are applicable to my changes as well. I will only update the iOS image accordingly.

Issues Fixed

Fixes #26049

Tested the behaviour in the following platforms

  • Android
  • Windows
  • iOS
  • Mac

Output Screenshot

Before After
BeforeFix.mov
AfterFix.mov

Copy link
Contributor

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

@dotnet-policy-service dotnet-policy-service bot added the community ✨ Community Contribution label Nov 22, 2024
@jsuarezruiz jsuarezruiz added platform/iOS 🍎 area-controls-shell Shell Navigation, Routes, Tabs, Flyout labels Nov 25, 2024
@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-shell Shell Navigation, Routes, Tabs, Flyout community ✨ Community Contribution platform/iOS 🍎
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ShellContent Title Does Not Update at Runtime
2 participants