Commit 4ab0f39
authored
Convert LayoutGridField to function components (rjsf-team#4807)
* fix typo
* LayoutGridField: extract static members
* LayoutGridField: all function components
* Improve types/inline docs
* Wrap `onFieldChange` in `useCallback`
* Fix parallel build issue caused by core/snapshot-tests dependency cycle
* Fix tests
* Add to CHANGELOG
* onFieldChange is actually unnecessary
* reviewer feedback
LayoutGridField:
- Deep compare memoize the fieldPathId
- Move helper functions in LayoutGridField
- Update docs comments in LayoutGridField
- Attach TEST_IDS to LayoutGridField function
Additionally:
- Update migration guide per rjsf-team#4808
* reviewer feedback
- Move LayoutGridFieldChildren component definition above usage
- LayoutGridFieldComponent": Simplify props destructure
- Fix issue with the nx dependency graph - dependencies were not tracked1 parent adeb182 commit 4ab0f39
File tree
6 files changed
+725
-724
lines changed- packages
- core
- src/components/fields
- test
- docs/docs/migration-guides
- snapshot-tests
6 files changed
+725
-724
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
| 47 | + | |
47 | 48 | | |
48 | 49 | | |
49 | 50 | | |
| |||
126 | 127 | | |
127 | 128 | | |
128 | 129 | | |
| 130 | + | |
| 131 | + | |
129 | 132 | | |
130 | 133 | | |
131 | 134 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
| 5 | + | |
12 | 6 | | |
13 | 7 | | |
14 | 8 | | |
| |||
0 commit comments