Skip to content

Conversation

Nuklon
Copy link

@Nuklon Nuklon commented Aug 23, 2025

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

Wpf Ui Gallery_hdSoXXq0GF

This focus looks ugly (accent border goes through the side and it doesn't cover the entire picker), and as DatePicker doesn't seem to have a working FocusVisualStyle, remove the focus style there completely, and allow the user to go to the calendar button with TAB as well (this now DOES focus).

image

I also fixed the missing dropshadow for calender picker:
image

@Nuklon Nuklon requested a review from pomianowski as a code owner August 23, 2025 11:59
@github-actions github-actions bot added controls Changes to the appearance or logic of custom controls. styles Topic is related to styles PR Pull request release labels Aug 23, 2025
@Nuklon
Copy link
Author

Nuklon commented Aug 23, 2025

Depends on #1472.

@Nuklon
Copy link
Author

Nuklon commented Aug 24, 2025

With some converters we can still get a proper focus style of the textbox.

image image image

Updated the PR. Using TAB still works now, which is nice improvement over the original version.

@Nuklon
Copy link
Author

Nuklon commented Aug 25, 2025

We can actually make it much simpler now by just reusing the default TextBox style. This also fixes the darker focus border at the bottom as shown in the above screenshot (which is what I wanted to investigate in the first place).

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
controls Changes to the appearance or logic of custom controls. dotnet PR Pull request release styles Topic is related to styles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants