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

Add natural number money input demo #1493

Merged
merged 4 commits into from
Jun 29, 2017

Conversation

cncolder
Copy link
Contributor

@cncolder cncolder commented Jun 27, 2017

添加一个例子,限制 money 键盘输入以0开头或包括小数点的数字,只能输入正整数(自然数)。

快照里面 stepper max min 从数值变成 Infinity 了,原因不明,所以没更新快照。

First of all, thank you for your contribution! :-)

Please makes sure that these checkboxes are checked before submitting your PR, thank you!

  • Make sure that you follow antd's code convention.
  • Run npm run lint and fix those errors before submitting in order to keep consistent code style.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

Extra checklist:

if isBugFix :

  • Make sure that you add at least one unit test for the bug which you had fixed.

elif isNewFeature :

  • Update API docs for the component.
  • Update/Add demo to demonstrate new feature.
  • Update TypeScript definition for the component.
  • Add unit tests for the feature.

This change is Reviewable

@mention-bot
Copy link

@cncolder, thanks for your PR! By analyzing the history of the files in this pull request, we identified @pingan1927 to be a potential reviewer.

@cncolder cncolder requested a review from pingan1927 June 27, 2017 21:45
@cncolder cncolder changed the title add int only money input demo Add natural number money input demo Jun 27, 2017
@paranoidjk
Copy link
Contributor

snapshot的变更,更新node modules再跑,rc-input-number升级了

@cncolder
Copy link
Contributor Author

snapshot updated

@codecov
Copy link

codecov bot commented Jun 28, 2017

Codecov Report

Merging #1493 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1493   +/-   ##
=======================================
  Coverage   65.06%   65.06%           
=======================================
  Files         229      229           
  Lines        2290     2290           
  Branches      701      701           
=======================================
  Hits         1490     1490           
  Misses        799      799           
  Partials        1        1
Flag Coverage Δ
#rn 79.81% <ø> (ø) ⬆️
#web 64.32% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 07288df...0028afe. Read the comment docs.

@paranoidjk paranoidjk merged commit 9cc1ba6 into ant-design:master Jun 29, 2017
lixiaoyang1992 pushed a commit to lixiaoyang1992/ant-design-mobile that referenced this pull request Apr 26, 2018
* add int only money input demo

* fix focus

* rename int to natural

* update snapshot
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.

3 participants