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

[UWP] Fix Button Disabled VisualState #13148

Merged
merged 3 commits into from
May 11, 2021
Merged

[UWP] Fix Button Disabled VisualState #13148

merged 3 commits into from
May 11, 2021

Conversation

jsuarezruiz
Copy link
Contributor

@jsuarezruiz jsuarezruiz commented Dec 15, 2020

Description of Change

Fix Button Disabled VisualState on UWP.

Issues Resolved

API Changes

None

Platforms Affected

  • UWP

Behavioral/Visual Changes

None

Before/After Screenshots

fix12984

Testing Procedure

Launch Core Gallery and navigate to the issue 12984. If the background color of the disabled Button is pink, the test has passed.

PR Checklist

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

@rubiomiguel06
Copy link

rubiomiguel06 commented Feb 19, 2021

Any news on when will this be merged to XF?

@abanoub-labib
Copy link

Is this going to be merged soon please?

@mhrastegari
Copy link
Contributor

When this will be merged? :(

@jsuarezruiz
Copy link
Contributor Author

@rachelkang Could you review this PR?. Thanks!

@rachelkang rachelkang self-requested a review May 10, 2021 18:07
@rachelkang
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@rachelkang rachelkang left a comment

Choose a reason for hiding this comment

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

Would be great if we could add some tests, but otherwise LGTM!

@jsuarezruiz jsuarezruiz merged commit b85c28e into 5.0.0 May 11, 2021
@jsuarezruiz jsuarezruiz deleted the fix-12984 branch May 11, 2021 08:23
@MagicAndre1981
Copy link
Contributor

SR4 broke disabling Buttons via VisualStateManager for me, it was working in 4.8 and till SR3

@MagicAndre1981
Copy link
Contributor

@jsuarezruiz any comments on the issue? Your "Fix" broke it when it was working before.

@MagicAndre1981
Copy link
Contributor

@jsuarezruiz any comments on the issue? It was also reported by a different user in #14428

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug][UWP] VisualState "Disabled" not working on Button
6 participants