Skip to content

Conversation

@Davide244
Copy link

Fix issue #2182

PR fixes issue #2182, which has multiple unreported duplicates at the time of writing.

Description of the changes:

The NumberPad now looks at the parent grid, grabs the width of the columns, and applies it to its own columns. This eliminates the misaligned buttons in NumberPad on window resize.

How changes were validated:

The changes can be observed in the UI by looking at it. (This has been checked on all available DPI presets in the windows settings.)

  • UI before the changes
image
  • UI after the changes
image

Testing:

Specify how you tested your changes (i.e. manual/ad-hoc testing, automated testing, new automated tests added)-->

  • The UI was tested manually.
  • Provided automated UI tests were run.
  • Test Results before code changes
image
  • Test Results after code changes
image

…s parent

The NumberPad now looks at the parent grid, grabs the width of the columns, and applies it to its own columns. This eliminates the misaligned buttons in NumberPad on window resize.
@Davide244
Copy link
Author

@microsoft-github-policy-service agree

@Georgios1999
Copy link

Thank you, our lord and savior, for fixing the damned misalignment. I can't believe it took 218 lines of code holy hell but Microsoft should hire you man

Copy link

@Georgios1999 Georgios1999 left a comment

Choose a reason for hiding this comment

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

Perfect. Just perfect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants