Skip to content

[iOS] Fix ShellContent Title Does Not Update at Runtime #26062

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

Merged
merged 14 commits into from
Feb 13, 2025

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.

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).

@devanathan-vaithiyanathan devanathan-vaithiyanathan marked this pull request as ready for review November 29, 2024 09:56
@devanathan-vaithiyanathan devanathan-vaithiyanathan requested a review from a team as a code owner November 29, 2024 09:56
@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis jfversluis requested a review from PureWeen December 6, 2024 13:22
@PureWeen PureWeen added this to the .NET 9 SR3 milestone Dec 6, 2024
@sheiksyedm sheiksyedm added the partner/syncfusion Issues / PR's with Syncfusion collaboration label Dec 6, 2024
@PureWeen PureWeen modified the milestones: .NET 9 SR3, .NET 9 SR4 Jan 5, 2025
@jsuarezruiz
Copy link
Contributor

jsuarezruiz commented Jan 10, 2025

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jsuarezruiz
Copy link
Contributor

@devanathan-vaithiyanathan Could you fix the conflict? Thanks in advance.

@devanathan-vaithiyanathan
Copy link
Contributor Author

devanathan-vaithiyanathan commented Jan 16, 2025

@devanathan-vaithiyanathan Could you fix the conflict? Thanks in advance.

@jsuarezruiz I have resolved the conflicts

@PureWeen
Copy link
Member

/rebase

Copy link
Member

@PureWeen PureWeen left a comment

Choose a reason for hiding this comment

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

Can you add a couple scenarios here

  • Add a shellcontent dynamically
  • you're able to update the title of that new tab
  • remove that shellcontent
  • you're able to update the title of an exising tab

@devanathan-vaithiyanathan
Copy link
Contributor Author

Can you add a couple scenarios here

  • Add a shellcontent dynamically
  • you're able to update the title of that new tab
  • remove that shellcontent
  • you're able to update the title of an exising tab

As per suggestion, I have modified the test. Please let me know if you have any concerns

@jfversluis jfversluis requested a review from PureWeen January 31, 2025 15:29
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

PureWeen commented Feb 4, 2025

/rebase

@PureWeen PureWeen modified the milestones: .NET 9 SR4, .NET 9 SR5 Feb 4, 2025
@jsuarezruiz
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@PureWeen
Copy link
Member

PureWeen commented Feb 9, 2025

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@jfversluis jfversluis dismissed stale reviews from jsuarezruiz and PureWeen February 13, 2025 13:18

Feedback addressed

@jfversluis
Copy link
Member

Failing test unrelated

@jfversluis jfversluis merged commit 3f9e537 into dotnet:main Feb 13, 2025
122 of 124 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 2025
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 community ✨ Community Contribution partner/syncfusion Issues / PR's with Syncfusion collaboration platform/iOS 🍎
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[iOS/Mac] ShellContent Title Does Not Update at Runtime
6 participants