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

Form Component #151

Closed
wants to merge 37 commits into from
Closed

Form Component #151

wants to merge 37 commits into from

Conversation

qial
Copy link
Collaborator

@qial qial commented Aug 19, 2020

What does this implement/fix? Explain your changes.
Will solve #136 by updating the form component to be more useable.

Copy link
Contributor

@reusr1 reusr1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • we are missing a way to tell the author that the form schema is invalid while working on the schema
  • we need better samples
  • general code cleanup as mentioned in the review

fragments/form/model.json Outdated Show resolved Hide resolved
fragments/form/sample-contact.json Outdated Show resolved Hide resolved
fragments/form/model.json Show resolved Hide resolved
fragments/form/sample.json Outdated Show resolved Hide resolved
fragments/form/template.html Outdated Show resolved Hide resolved
fragments/form/template.vue Outdated Show resolved Hide resolved
fragments/form/template.vue Outdated Show resolved Hide resolved
fragments/form/template.vue Outdated Show resolved Hide resolved
fragments/form/template.vue Outdated Show resolved Hide resolved
fragments/form/template.vue Outdated Show resolved Hide resolved
fragments/form/model.json Outdated Show resolved Hide resolved
@smcgrath0
Copy link
Collaborator

Added styles to this component. And removed depublicate submit button in all-inputs schema.

form-test

@smcgrath0
Copy link
Collaborator

Component now allows button to be aligned left/center/right, and ability to change size

button-form-test

@qial qial linked an issue Sep 4, 2020 that may be closed by this pull request
@qial qial changed the title Form Component Updates Form Component and Datalist Component Sep 4, 2020
@qial qial changed the title Form Component and Datalist Component Form Component Sep 4, 2020
@qial
Copy link
Collaborator Author

qial commented Sep 5, 2020

I am closing this in favor of directly pulling in the issues/156 branch in the pull request 162

@qial qial closed this Sep 5, 2020
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.

form component
3 participants