-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Handle NumericUpDown percentage StringFormat correctly #3678
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@punker76 I'll have a look at this and let you know if I have any issues with the new implementation.
Hi @punker76 some more thoughts on your PR:
Happy coding and a 💯 times thanks. |
Good morning @punker76 ,
one addition: if we change the basetype to Happy coding |
97258c7
to
920bd5b
Compare
The current behavior for e.g. percentage text input with StringFotmat like P0 or {0:0.00%} converts the input one-to-one to the value: 100% = 100 This is not what the user expects, because the .Net default works different: 100% = 1 So this PR changes this behavior to the .Net default one.
920bd5b
to
f4d8680
Compare
Describe the changes you have made to improve this project
The current behavior for e.g. percentage text input with StringFotmat like P0 or {0:0.00%} converts the input one-to-one to the value:
100% = 100
This is not what the user expects, because the .Net default works different:
100% = 1
So this PR changes this behavior to the .Net default one.
Unit test
Additional context
Closed Issues
Closes #3376
/cc @mgnslndh @timunie