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

ColorPicker Simple Theme Fixes #16654

Merged

Conversation

robloo
Copy link
Contributor

@robloo robloo commented Aug 10, 2024

What does the pull request do?

Fixes a few issues with the ColorView and ColorPicker in simple theme. I was always hoping simple theme was going away so the initial theme was only functional. This fixes a few issues though which is important now that it's used in dev tools and things.

What is the current behavior?

  1. You have to click the tab icon to change tabs. Instead, anywhere should be allowed
  2. The padding in the hex color input TextBox is bad
  3. NumericUpDown shows a vertical scrollbar when it shouldn't

image

What is the updated/expected behavior with this PR?

  1. You have to click the tab icon to change tabs. Instead, anywhere should be allowed
    • This is really an issue with the TabControl theme but I'm not fixing it there to avoid breaking other code. I'm just adding a transparent background in the color picker templates instead.
  2. The padding in the hex color input TextBox is bad
    • This is really an issue with the underlying TextBox IMO but I'm not fixing it generically as it would be breaking. Instead, I'm just copying the better padding from the Fluent theme to use here. The Fluent theme is designed for 32px height controls already.
  3. NumericUpDown shows a vertical scrollbar when it shouldn't
    • Again, this is an issue of TextBox in the simple theme. I didn't feel like addressing the issue there. However, it was fixed for all NumericUpDown controls. These controls should always be one line so don't require vertical scrolling. Therefore, vertical scrolling was just disabled across the board for this control.

image

How was the solution implemented (if it's not obvious)?

As described above.

Checklist

Breaking changes

None

Obsoletions / Deprecations

None

Fixed issues

Fixes #16497

@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0051151-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@cla-avalonia
Copy link
Collaborator

cla-avalonia commented Aug 10, 2024

  • All contributors have signed the CLA.

@robloo
Copy link
Contributor Author

robloo commented Aug 10, 2024

@cla-avalonia agree

Copy link
Member

@maxkatz6 maxkatz6 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you.

@avaloniaui-bot
Copy link

You can test this PR using the following package version. 11.2.999-cibuild0051257-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@maxkatz6 maxkatz6 added this pull request to the merge queue Aug 15, 2024
Merged via the queue into AvaloniaUI:master with commit 9fe3415 Aug 15, 2024
10 checks passed
@robloo robloo deleted the robloo/colorpicker-simple-theme-fix branch August 15, 2024 10:37
@maxkatz6 maxkatz6 added bug and removed enhancement labels Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ColorView - broken style in Simple theme
5 participants