-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fixed FontImageSource icon color does not change in the TabbedPage when dynamically updated. #27742
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
…e when dynamically updated.
Hey there @NirmalKumarYuvaraj! Thank you so much for your PR! Someone from the team will get assigned to your PR shortly and we'll get it reviewed. |
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.
@jsuarezruiz , Yes, this is the test case I am referring to. Once this gets merged, I will update the button click logic to change the color instead of setting a new FontImageSource. |
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
…en dynamically updated. (#27742) * Fixed The FontImageSource icon color does not change in the TabbedPage when dynamically updated. * Updated code changes
…en dynamically updated. (dotnet#27742) * Fixed The FontImageSource icon color does not change in the TabbedPage when dynamically updated. * Updated code changes
Issue Details
The FontImageSource Icon color of the TabbedPage is not updated at run time.
Root Cause
The SourceChanged event is not implemented for the IconImageSource in the Page.
Description of Change
Implementing a source changed event listener for IconImageSource in the ImageSource call back. When the ImageSource changes are detected, manually triggering PropertyChanged for IconImageSource.This propagates the change through the UI update mechanism, ensuring the TabbedPage icon reflects the new state.
Validated the behaviour in the following platforms
Issues Fixed
Fixes #27549
NOTE : This PR is dependent on the following PR's .
Android and iOS PR - #26757
Windows PR - #26888
TestCase - I will modify the testcase once the above PR's are merged.
Output
Before.mov
After.mov