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

Remove Input autoComplete restriction to allow more complex auto-filling behaviour #7699

Merged
merged 1 commit into from
Sep 22, 2017

Conversation

delesseps
Copy link
Contributor

@delesseps delesseps commented Sep 22, 2017

As well as on and off there are many other possible values such as email, username, new-password, current-password and credit card options to help the browser autofill.

References

@delesseps delesseps changed the title Remove autoComplete restriction to allow more complex auto-filling behaviour Remove Input autoComplete restriction to allow more complex auto-filling behaviour Sep 22, 2017
@codecov
Copy link

codecov bot commented Sep 22, 2017

Codecov Report

Merging #7699 into antd-3.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           antd-3.0   #7699   +/-   ##
========================================
  Coverage      82.1%   82.1%           
========================================
  Files           214     214           
  Lines          4453    4453           
  Branches       1342    1342           
========================================
  Hits           3656    3656           
  Misses          797     797
Impacted Files Coverage Δ
components/input/Input.tsx 87.93% <ø> (ø) ⬆️

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 df69320...4d54b0a. Read the comment docs.

@afc163 afc163 merged commit bb41151 into ant-design:antd-3.0 Sep 22, 2017
@delesseps delesseps deleted the antd-3.0 branch September 26, 2017 06:42
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.

2 participants