Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[iOS, UWP] Background Brush issues #14614

Merged
merged 6 commits into from
Oct 19, 2021
Merged

[iOS, UWP] Background Brush issues #14614

merged 6 commits into from
Oct 19, 2021

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Sep 14, 2021

Description of Change

Add an implicit conversion from Color to SolidColorBrush to avoid issues where the binding value use a wrong type.

issue-14613

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

None

Before/After Screenshots

Not applicable

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@StephaneDelcroix
Copy link
Member

Could you update the PR description and title? this does a bit more than adding a sample, it adds an op_implicit, and that's what fixes it. Or am I misreading something ?

@jsuarezruiz jsuarezruiz changed the title [iOS] SolidColorBrush on Frame Background issue [iOS, UWP] Background Brush issues Sep 15, 2021
@jsuarezruiz jsuarezruiz added this to Issues in Progress in 5.0.0 SR6 (Planning) - Target Date Oct. 13th via automation Sep 15, 2021
@jsuarezruiz
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jsuarezruiz
Copy link
Contributor Author

@StephaneDelcroix You are right, updated the description.

@jfversluis jfversluis added this to PR Build/Test Failures in 5.0.0 SR7 (Planning) - Target Date Nov. 10th Oct 18, 2021
@jfversluis jfversluis moved this from PR Build/Test Failures to PR Needs Review in 5.0.0 SR7 (Planning) - Target Date Nov. 10th Oct 19, 2021
@jfversluis jfversluis merged commit 3dcfa50 into 5.0.0 Oct 19, 2021
5.0.0 SR7 (Planning) - Target Date Nov. 10th automation moved this from PR Needs Review to Done Oct 19, 2021
@jfversluis jfversluis deleted the fix-14613 branch October 19, 2021 10:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

[Bug] Frame Background property not working [Bug] Entry Background doesn't work to set color
3 participants