All CLA requirements met.

Codecov Report

Merging #4127 into master will increase coverage by 29.99531%.
The diff coverage is 100.00000%.

@@                 Coverage Diff                  @@
##              master       #4127          +/-   ##
====================================================
+ Coverage   68.12157%   98.11688%   +29.99530%     
====================================================
  Files           1482         494         -988     
  Lines         508855      258454      -250401     
  Branches       41468        4489       -36979     
====================================================
- Hits          346640      253587       -93053     
+ Misses        155976        4105      -151871     
+ Partials        6239         762        -5477     

Flags with carried forward coverage won't be shown. Click here to find out more.

@RussKie, actually, no. That is my mistake in #2382. But we can't set true here just like that because a control with properties Enabled=false or Visible=false can't be focused. So I would prefer to use _owningControl.CanSelect property here. But ControlAccessibleObject already implements it, so @albertvaka, we can just remove IsKeyboardFocusable definitions from DomainUpDown/NumericUpdown AccessibleObjects.
@albertvaka, could you please check if it is correct? And also please add unit tests for it if it is possible.

Please add unit tests. Thank you.

@msftbot merge if @vladimir-krestov approves

Hello @RussKie!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

• I'll only merge this pull request if it's approved by @vladimir-krestov

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".

The code is fine, I need to do some tests.

All right! Thank you 😊

[Unrelated to this change] While testing this PR I found a small change was needed to the arcade repo for NativeTests to build and I sent a PR that's not getting reviewed. If you folks can give it a look, here it is: dotnet/arcade#6462

It's installed separately and I'm using the latest one. Maybe some versions of CMake accept an empty source dir and default to . and because of that it was working by chance before? In any case, I think the MSBuild file is actually missing that dependency so the source dir gets properly defined.