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

Fields component and Form examples #94

Merged
merged 7 commits into from
Oct 27, 2015

Conversation

levithomason
Copy link
Member

Adds form examples and the <Fields /> component.

EDIT
These turned out to be non issues. The doc site shows usage:
Fixes #36 #38 #92.

@levithomason levithomason changed the title Dynamic input wrapper Fields component and Form examples Oct 27, 2015
const required = item.required && <span className='ui mini red circular label'>required</span>;
return <div>{item.name} {required}</div>;
const required = item.required && <span className='ui empty mini red circular label' />;
return <code>{item.name} {required}</code>;
Copy link
Member Author

Choose a reason for hiding this comment

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

Format props table to use monospace font and red dot for required.

description='If a form is in an error state, it will automatically show any error message blocks.'
examplePath='collections/Form/States/FormFieldErrorExample'
/>
</ExampleSection>
Copy link
Member Author

Choose a reason for hiding this comment

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

image

@eanplatter
Copy link
Contributor

Makes sense 🍂

levithomason added a commit that referenced this pull request Oct 27, 2015
…apper

Fields component and Form examples
@levithomason levithomason merged commit dbe6f7c into master Oct 27, 2015
@levithomason levithomason deleted the feature/dynamic-input-wrapper branch October 27, 2015 22:07
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