SplitButton Background color not updating when IsEnabled is false

[capegreg]

Describe the bug

The SplitButton Background color does not update to disabled color as a standard Button background color will change when the button IsEnabled is false.

Steps to reproduce the bug

Add the xmlns:muxc="using:Microsoft.UI.Xaml.Controls" namespace to a XAML page.

Add a SplitButton to XAML page

<muxc:SplitButton Background="#FFEAEAE0" Content="I am a SplitButton" Foreground="Black" IsEnabled="False" Height="48" Width="177" />

Add a Button to XAML page

View the page in designer window or run the application.

Expected behavior

The SplitButton background color should be the same as the Button background color since IsEnabled for both are "False".

Screenshots

NuGet package version

No response

Windows app type

• UWP
• Win32

Device form factor

Desktop

Windows version

May 2020 Update (19041)

Additional context

No response

[StephenLPeters]

We have SplitButtonDisabledBackground resources, but they are simply not used. The control doesn't have a disabled state that could use them at all...

[marcelwgn]

@StephenLPeters Is it correct to add the appropriate visual state then and set that resource? Mind if I work on that then?

[capegreg]

Update. I added the following resource to get disabled style.

Background="{ThemeResource ButtonBackgroundDisabled}"
BorderBrush="{ThemeResource ButtonBorderBrushDisabled}"

[StephenLPeters]

@chingucoding I think so, we will want to test it pretty thoroughly though, the state manager seems complicated to me...

[ghost]

🎉This issue was addressed in #6138, which has now been successfully released as Microsoft.UI.Xaml v2.8.0-prerelease.220413001.:tada:

Handy links:

• Release Notes
• Official Nuget Release

[ghost]

🎉This issue was addressed in #6138, which has now been successfully released as Microsoft.UI.Xaml v2.8.0-prerelease.220712001.:tada:

Handy links:

• Release Notes
• Official Nuget Release

[ghost]

🎉This issue was addressed in #6138, which has now been successfully released as Microsoft.UI.Xaml v2.8.0.:tada:

Handy links:

• Release Notes
• Official Nuget Release

[ghost]

🎉This issue was addressed in #6138, which has now been successfully released as Microsoft.UI.Xaml v2.8.2.:tada:

Handy links:

• Release Notes
• Official Nuget Release