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

ComboBox disable background color is different on Skia #18590

Open
ArchieCoder opened this issue Oct 25, 2024 · 3 comments
Open

ComboBox disable background color is different on Skia #18590

ArchieCoder opened this issue Oct 25, 2024 · 3 comments
Assignees
Labels
difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI kind/bug Something isn't working project/styling 👔 Categorizes an issue or PR as relevant to element styling

Comments

@ArchieCoder
Copy link
Contributor

Current behavior

On WinUI, the disabled background works as expected.

Side bug: when using a Style, Skia does not uses the brushes that I specify.

image

Expected behavior

Same output as WinUI

How to reproduce it (as minimally and precisely as possible)

ComboBox.zip

Workaround

No response

Works on UWP/WinUI

Yes

Environment

No response

NuGet package version(s)

No response

Affected platforms

Skia (WPF), Skia (Linux X11), Skia (macOS)

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response

@ArchieCoder ArchieCoder added difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. kind/bug Something isn't working triage/untriaged Indicates an issue requires triaging or verification labels Oct 25, 2024
@morning4coffe-dev morning4coffe-dev self-assigned this Oct 26, 2024
@morning4coffe-dev
Copy link
Member

@ArchieCoder, I managed to replicate the issue even with the latest dev versions (SDK 5.6.0-dev.51). I'll look into more details on why this might be happening. Did you test the same behavior with any other controls?

@ArchieCoder
Copy link
Contributor Author

@morning4coffe-dev I did a quick test with TextBox and using IsEnabled to false produces a different result. I have a feeling that testing IsEnabled=False with all the rest of the controls will lead to different results as well.

image

@MartinZikmund MartinZikmund added project/styling 👔 Categorizes an issue or PR as relevant to element styling difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI and removed triage/untriaged Indicates an issue requires triaging or verification difficulty/tbd Categorizes an issue for which the difficulty level needs to be defined. labels Oct 28, 2024
@ramezgerges
Copy link
Contributor

@morning4coffe-dev This might be related to #18724

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty/medium 🤔 Categorizes an issue for which the difficulty level is reachable with a good understanding of WinUI kind/bug Something isn't working project/styling 👔 Categorizes an issue or PR as relevant to element styling
Projects
None yet
Development

No branches or pull requests

4 participants