Break down CustomTableCell in two components and remove `usedIn` prop
Again, probably better reviewed in separate commits.
The first breaks down CustomTableCell
in two components: it does not yield immediate gain but allows to see more clearly the different usages of the ex-CustomTableCell
.
This allows notably to better adapt props to the actual use and laid the groundwork for the second commit that removes the usedIn
prop.
The usedIn
prop was used to
- check for invalidity of the field
- check if boolean and number inputs should be used for the field
value
- Conditionally set the
width
for the fieldname
.
This was solved by
- checking the invalidity in the caller
- adding a
allowBoolAndNumberInput
prop toValueTableCell
- add a
width
prop toNameTableCell
I expect to be able to remove width
and allowBoolAndNumberInput
in the future by reworking the abstractions but let's not jump the gun.
Edited by Loic Huder