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

NumericUpDown Culture problem #2149 #2199

Merged
merged 2 commits into from
Dec 2, 2015
Merged

NumericUpDown Culture problem #2149 #2199

merged 2 commits into from
Dec 2, 2015

Conversation

fredericoregateiro
Copy link
Contributor

Tested with different cultures and it seems to work just fine.
When the culture NumberDecimalSeparator is ',' (comma) the control replaces the '.' (period or decimal key) with the corrent NumberDecimalSeparator symbol.

All of the remaining functionality is the same.

@fredericoregateiro
Copy link
Contributor Author

@punker76 do you think this can get on the 1.2.0 release?

if (this.Culture != null)
{
if (textBox.Text.Contains(this.Culture.NumberFormat.NumberDecimalSeparator) == false)
{
Copy link
Member

Choose a reason for hiding this comment

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

It's better to use SpecificCultureInfo property nstead Culture.

@punker76
Copy link
Member

@fredericoregateiro I will not decide this alone...

/cc @xxMUROxx

@fredericoregateiro
Copy link
Contributor Author

So, any decision on this one?

@michaelmairegger
Copy link
Contributor

@fredericoregateiro sorry, I have forgotten to test this. I will give you then a feedback

@michaelmairegger
Copy link
Contributor

@punker76 I have tested it. I have found no problems.
In my opinion we can merge this PR which closes #2149

punker76 added a commit that referenced this pull request Dec 2, 2015
@punker76 punker76 merged commit 19d96fe into MahApps:master Dec 2, 2015
@punker76 punker76 added this to the 1.2.0 milestone Dec 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants