Bug #33238
[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
Associated revisions
History
#1
Updated by Jonathon Turel over 1 year ago
- Triaged changed from No to Yes
- Target version changed from Katello 4.2.0 to Katello 4.3.0
#2
Updated by The Foreman Bot over 1 year ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/9550 added
#3
Updated by The Foreman Bot over 1 year ago
- Fixed in Releases Katello 4.3.0 added
#4
Updated by Jeremy Lenz over 1 year ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|46d3cb1468ca468b46a61fdc0aca6de8edc92cae.
Fixes #33238 - Extend TableWrapper and MainTable (#9550)
to allow using PF TableComposable
Refactor ContentViewHistories to use composable table
update babel parser
add class for pf3 table overflow auto