-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[v6] Add react-bootstrap theme to replace bootstrap-4 (#3638) #3974
Conversation
"@rjsf/chakra-ui": "^5.14.2", | ||
"@rjsf/core": "^5.14.2", | ||
"@rjsf/fluent-ui": "^5.14.2", | ||
"@rjsf/fluentui-rc": "^5.14.2", | ||
"@rjsf/material-ui": "^5.14.2", | ||
"@rjsf/mui": "^5.14.2", | ||
"@rjsf/react-bootstrap": "file:../react-bootstrap", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This lets me build the playground without releasing the theme to NPM. We can update this to the version number once we release a v6 beta.
Converting to draft until we get CI working for rjsf-v6 branch |
@nickgros I guess the first thing we need to do is create the long-lived branch |
- Add react-bootstrap package and use react-bootstrap v2, corresponding to Bootstrap 5 - CheckboxesWidget: Remove deprecated prop `custom` - IconButton: Remove deprecated `block` prop - RangeWidget: Use `FormRange` component - SelectWidget: Use new FormSelect component, remove `bsPrefix` prop to achieve correct styling
CHANGELOG.md
Outdated
@@ -15,6 +15,21 @@ it according to semantic versioning. For example, if your PR adds a breaking cha | |||
should change the heading of the (upcoming) version to include a major version bump. | |||
|
|||
--> | |||
# 6.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# 6.0.0 | |
# 6.0.0-beta.1 |
packages/react-bootstrap/README.md
Outdated
|
||
### Built With | ||
|
||
- [react-jsonschema-form](https://github.com/mozilla-services/react-jsonschema-form/) | ||
- [Bootstrap-4](https://getbootstrap.com/docs/4.0/) | ||
- [Bootstrap-5](https://getbootstrap.com/docs/4.0/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update this to 5
"version": "5.15.0", | ||
"main": "dist/index.js", | ||
"module": "lib/index.js", | ||
"typings": "lib/index.d.ts", | ||
"description": "Bootstrap 4 theme, fields and widgets for react-jsonschema-form", | ||
"description": "Bootstrap 5 theme, fields and widgets for react-jsonschema-form, powered by react-bootstrap", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider calling this React Bootstrap
theme
"bootstrap-5", | ||
"rjsf-bootstrap-5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add react-bootstrap
, rjsf-react-bootstrap
|
||
## @rjsf/bootstrap-4 | ||
|
||
- Package has been replaced with `@rjsf/react-bootstrap`. `react-boostrap` v1 / Bootstrap 4 are no longer supported in RJSF v6. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, add this to the migration guide with breaking changes?
Fixes #3638
Add react-bootstrap package and use react-bootstrap v2, corresponding to Bootstrap 5
CheckboxesWidget: Remove deprecated prop
custom
IconButton: Remove deprecated
block
propRangeWidget: Use
FormRange
componentSelectWidget: Use new FormSelect component, remove
bsPrefix
prop to achieve correct stylingI'm updating documentation
I'm adding or updating code
npm run test:update
to update snapshots, if needed.I'm adding a new feature