Actions
Bug #33238
closed[SAT-1790] Extend TableWrapper and MainTable to allow using PF TableComposable
Description
https://patternfly-react.surge.sh/components/table#composable-basic
Let's start all our tables off on the right foot.
- no more formatters
- no more mixed JSX and logic
- no more guessing what your markup is going to look like, or looking in 4 different files
- no more 'rows' and 'cells' props
- keep data, logic, and markup neatly separate and organized
- that much easier to get started and probably cause less bugs
This work will consist of
- add a 'composable' prop to <TableWrapper>
- if composable prop is passed, MainTable returns <TableComposable> instead of <Table> and doesn't accept rows or cells props-- so you can do either <TableWrapper rows={...} cells={...} /> or <TableWrapper composable > ... </TableWrapper>
- all of the TableComposable benefits listed above
- we can reuse the pagination and search parts of TableWrapper
Updated by Jonathon Turel about 3 years ago
- Target version changed from Katello 4.2.0 to Katello 4.3.0
- Triaged changed from No to Yes
Updated by The Foreman Bot about 3 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/9550 added
Updated by The Foreman Bot about 3 years ago
- Fixed in Releases Katello 4.3.0 added
Updated by Jeremy Lenz about 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|46d3cb1468ca468b46a61fdc0aca6de8edc92cae.
Actions