-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fix(module:overlay): format number InvariantCulture #1956
fix(module:overlay): format number InvariantCulture #1956
Conversation
In cultures where decimal separator is ',' overlay would be ill-positioned Resolves ant-design-blazor#1955
Thanks for contribute this. |
Codecov Report
@@ Coverage Diff @@
## master #1956 +/- ##
==========================================
+ Coverage 25.44% 27.30% +1.85%
==========================================
Files 487 510 +23
Lines 31942 24359 -7583
Branches 0 233 +233
==========================================
- Hits 8129 6651 -1478
+ Misses 23813 17672 -6141
- Partials 0 36 +36
Continue to review full report at Codecov.
|
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.
LGTM
In cultures where decimal separator is ',' overlay would be ill-positioned Resolves #1955 Co-authored-by: James Yeung <shunjiey@hotmail.com>
…or#1956) In cultures where decimal separator is ',' overlay would be ill-positioned Resolves ant-design-blazor#1955 Co-authored-by: James Yeung <shunjiey@hotmail.com>
In cultures where decimal separator is ',' overlay would be ill-positioned Resolves #1955 Co-authored-by: James Yeung <shunjiey@hotmail.com>
In cultures where decimal separator is ',' overlay would be ill-positioned Resolves #1955 Co-authored-by: James Yeung <shunjiey@hotmail.com>
In cultures where decimal separator is ',' overlay would be ill-positioned Resolves #1955 Co-authored-by: James Yeung <shunjiey@hotmail.com>
In cultures where decimal separator is ',' overlay would be ill-positioned
🤔 This is a ...
🔗 #1955
💡 Background and solution
If user has locale with ',' as decimal separator and the calculated left / top values have a decimal part the resulting formatted value will be not be parsable by browser:
📝 Changelog
☑️ Self Check before Merge