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 autocapitalize prop to UiTextbox #501

Merged
merged 3 commits into from
Dec 31, 2021

Conversation

Lewiscowles1986
Copy link
Contributor

@Lewiscowles1986 Lewiscowles1986 commented Jun 7, 2020

Fixes #500

Update: Now has documentation updates to showcase autocapitalize attribute

  • passes npm run lint
  • updates documentation
  • tested locally using yarn dev and opening http://127.0.0.1:9000

Really I think I'd prefer to pass through some form of key value map like in React so this doesn't need editing per-new attribute. Is that possible?
@Lewiscowles1986 Lewiscowles1986 marked this pull request as ready for review June 7, 2020 07:04
@JosephusPaye
Copy link
Owner

Looks good. Will merge for next release. Thanks!

@Lewiscowles1986
Copy link
Contributor Author

Haha. Any updates on merge. If I remember correctly, this is how I came to the repo. Kolibri uses / used this and as well as patching their local fork, I wanted to offer to upstream this after one of their users or devs raised an issue regarding autocapitalize on mobile.

@JosephusPaye JosephusPaye changed the title Autocapitalize attribute support Add autocapitalize prop to UiTextbox Dec 31, 2021
@JosephusPaye JosephusPaye merged commit 52a5b8e into JosephusPaye:next Dec 31, 2021
@JosephusPaye
Copy link
Owner

Thanks!

@JosephusPaye
Copy link
Owner

Released in v1.3.2

@Lewiscowles1986
Copy link
Contributor Author

Well Happy New Year to you and thanks.

@JosephusPaye
Copy link
Owner

Happy New Year! :)

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.

Support for autocapitalize property
2 participants