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

feat(react): Adds FieldWrap component #498

Merged
merged 4 commits into from
Jan 7, 2022
Merged

feat(react): Adds FieldWrap component #498

merged 4 commits into from
Jan 7, 2022

Conversation

schne324
Copy link
Member

@schne324 schne324 commented Jan 7, 2022

No description provided.

@schne324 schne324 marked this pull request as draft January 7, 2022 06:40
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2022

Preview branch generated at https://field-wrap.d1gko6en628vir.amplifyapp.com

}
}}
/>
<FieldWrap>
Copy link
Member Author

@schne324 schne324 Jan 7, 2022

Choose a reason for hiding this comment

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

resolves a11y issues in #386

}

/* reset absurd browser-default legend styles */
.Panel legend {
Copy link
Member Author

Choose a reason for hiding this comment

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

I've made it so the legend matches the exact styling of the Panel__Heading elements

}
}}
/>
<FieldWrap>
Copy link
Member Author

Choose a reason for hiding this comment

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

resolves a11y issues in #386

@@ -15,7 +16,8 @@ export default class RadioGroupDemo extends Component {
render() {
const { value } = this.state;
return (
<div>
<FieldWrap>
Copy link
Member Author

Choose a reason for hiding this comment

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

resolves a11y issues in #386

@@ -21,7 +22,8 @@ const SelectDemo = () => {
const [currentValue, setCurrentValue] = useState('Maybe');

return (
<div className="SelectDemo">
<FieldWrap className="SelectDemo">
Copy link
Member Author

Choose a reason for hiding this comment

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

resolves a11y issues in #386

@schne324 schne324 marked this pull request as ready for review January 7, 2022 06:57
@schne324 schne324 requested review from thuey and scurker January 7, 2022 16:06
Copy link
Collaborator

@thuey thuey left a comment

Choose a reason for hiding this comment

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

lgtm!

@schne324 schne324 merged commit 10f975c into develop Jan 7, 2022
@schne324 schne324 deleted the field-wrap branch January 7, 2022 17:59
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2022

Preview branch generated at https://field-wrap.d1gko6en628vir.amplifyapp.com

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