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

[AnimatedIcon] Fix issue where setting source property would not resize AnimatedIcon properly #6676

Merged

Conversation

marcelwgn
Copy link
Contributor

Description

Issue was that the layout was not invalidated when setting the source property.

Motivation and Context

Closes #6659

How Has This Been Tested?

Added new API test

Screenshots (if appropriate):

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Feb 3, 2022
@StephenLPeters
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@StephenLPeters
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@StephenLPeters StephenLPeters added area-AnimatedIcon team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Feb 28, 2022
@StephenLPeters StephenLPeters merged commit fe17f47 into microsoft:main Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-AnimatedIcon team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AnimatedIcon does not dirty layout when Source changes.
2 participants