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

feat: Add class name for the input element #146

Merged
merged 4 commits into from Nov 4, 2019

Conversation

Amwam
Copy link
Contributor

@Amwam Amwam commented Feb 10, 2019

Allows the addition of a custom class to the input element

The typescript definitions we also failing to pass the lint (and were possibly a TS v1 definition?), so I've imported from Definitely Typed, and updated.

@codecov
Copy link

codecov bot commented Feb 10, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #146   +/-   ##
=======================================
  Coverage   91.12%   91.12%           
=======================================
  Files           7        7           
  Lines         383      383           
  Branches       95       95           
=======================================
  Hits          349      349           
  Misses         34       34
Impacted Files Coverage Δ
src/TimePicker.jsx 89.1% <ø> (ø) ⬆️

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 9b0a7e6...baf76e5. Read the comment docs.

@jcleefw
Copy link

jcleefw commented Aug 5, 2019

This is what i'm looking for. Can we please merge this in?

@jcleefw
Copy link

jcleefw commented Aug 5, 2019

Think there's broken test with the extra spacing in the classname. Would you mind fixing it @Amwam. Thanks

@Amwam
Copy link
Contributor Author

Amwam commented Aug 6, 2019

@jcleefw Thanks, I've updated to fix the tests

@Amwam Amwam changed the title Add class name for the input element feat: Add class name for the input element Aug 6, 2019
@afc163 afc163 merged commit e77b310 into react-component:master Nov 4, 2019
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.

None yet

3 participants