-
Notifications
You must be signed in to change notification settings - Fork 1.8k
[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
Conversation
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. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
This reverts commit ae6a031.
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
src/Controls/src/Core/Compatibility/Handlers/Shell/iOS/ShellSectionRootHeader.cs
Show resolved
Hide resolved
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
@devanathan-vaithiyanathan Could you fix the conflict? Thanks in advance. |
@jsuarezruiz I have resolved the conflicts |
/rebase |
There was a problem hiding this 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
As per suggestion, I have modified the test. Please let me know if you have any concerns |
Azure Pipelines successfully started running 3 pipeline(s). |
/rebase |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
Azure Pipelines successfully started running 3 pipeline(s). |
Feedback addressed
Failing test unrelated |
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
Output Screenshot
BeforeFix.mov
AfterFix.mov