Skip to content

Add wrapper element around required asterisk in TitleField#1001

Merged
glasserc merged 1 commit intorjsf-team:masterfrom
domharrington:feature/title-field-required-element
Nov 9, 2018
Merged

Add wrapper element around required asterisk in TitleField#1001
glasserc merged 1 commit intorjsf-team:masterfrom
domharrington:feature/title-field-required-element

Conversation

@domharrington
Copy link
Contributor

Reasons for making this change

This is so that it can be separately styled easily via CSS.
It is already done this way in https://github.com/mozilla-services/react-jsonschema-form/blob/aadbbe9701c4db9df761e86f0280e1ecafc509f8/src/components/fields/SchemaField.js#L59

Checklist

  • I'm adding or updating code
    • I've added and/or updated tests
    • I've updated docs if needed
    • I've run npm run cs-format on my branch to conform my code to prettier coding style

domharrington pushed a commit to readmeio/api-explorer that referenced this pull request Aug 7, 2018
Relies on rjsf-team/react-jsonschema-form#1001 which
will hopefully get merged in
@glasserc glasserc merged commit 0c533b3 into rjsf-team:master Nov 9, 2018
@glasserc
Copy link
Contributor

glasserc commented Nov 9, 2018

Thanks!

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