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 mulit format support to TimePicker #133

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

NathanBP
Copy link

Allows format prop to allow an array of formats. The control then uses the array of formats when parsing a date from the input. The first value of the array determines the displayed format.

An example use case is that a user may want to enter a time as hmm but display as h:mm a.

@NathanBP
Copy link
Author

Test which is failing is already failing in master branch. Once this is fixed this pull request will pass the required checks.

@NathanBP
Copy link
Author

ant design issue link

@chenshuai2144
Copy link

great

@codecov
Copy link

codecov bot commented Jan 3, 2019

Codecov Report

Merging #133 into master will increase coverage by 0.49%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #133      +/-   ##
========================================
+ Coverage    89.5%    90%   +0.49%     
========================================
  Files           7      8       +1     
  Lines         381    390       +9     
  Branches       93     93              
========================================
+ Hits          341    351      +10     
+ Misses         40     39       -1
Impacted Files Coverage Δ
src/Panel.jsx 95.23% <ø> (ø) ⬆️
src/TimePicker.jsx 87.64% <100%> (ø) ⬆️
src/Combobox.jsx 93.9% <100%> (+0.15%) ⬆️
src/util/index.js 100% <100%> (ø)
src/Header.jsx 86.36% <100%> (+0.97%) ⬆️

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 c8cb160...691270c. Read the comment docs.

@tonihelminen
Copy link

Could this implemented feature be taken into use? It would be helpful at least in some countries around the world.

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