Skip to content

feat: added type definitions and textarea component #87

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

Closed
wants to merge 3 commits into from
Closed

feat: added type definitions and textarea component #87

wants to merge 3 commits into from

Conversation

pixelmund
Copy link
Contributor

@pixelmund pixelmund commented Feb 3, 2021

Those are the type definitions for svelte-forms-lib, there might be some optimizations but it's a good starting point.

Svelte currently does not support generics in components, there are some discussions around it but i couldn't find a solution.

Also i have added the textarea helper component as requested here: #86

fixes #34
fixes #81
implements #86

@pixelmund pixelmund changed the title feat: added type definitions feat: added type definitions and textarea component Feb 3, 2021
@pixelmund pixelmund closed this Feb 3, 2021
@pixelmund pixelmund deleted the branch tjinauyeung:master February 3, 2021 19:14
@pixelmund pixelmund deleted the master branch February 3, 2021 19:14
@pixelmund pixelmund restored the master branch February 3, 2021 19:15
@pixelmund pixelmund reopened this Feb 3, 2021
@pixelmund pixelmund closed this Feb 3, 2021
@pixelmund pixelmund deleted the master branch February 3, 2021 19:18
@larrybotha
Copy link
Collaborator

larrybotha commented Feb 3, 2021

@pixelmund thanks for the PR! What made you close it? Would be much appreciated if you submitted this (although I'd ask that you submit the TS definitions in a different PR from the textarea)

EDIT: I've added a PR for the textarea component here, along with adding test configs for components: #88

@pixelmund
Copy link
Contributor Author

I will reopen it later today with the type definitions only.

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.

Project updates Typescript defs?
2 participants