fix(table-tree-view): reset table tree view th header cell min/max-width, width , and display#6868
Conversation
|
Preview: https://patternfly-pr-6868.surge.sh A11y report: https://patternfly-pr-6868-a11y.surge.sh |
|
adding @lboehling and @andrew-ronaldson. @mattnolting please fill in any missing details, but I believe you made a change to the table in a previous PR to keep the table column header visible for the first column in a tree table on mobile. This PR just fixes a visual bug with that, but I don't think design had reviewed the initial change, so I want to make sure this is how design wants to present the table. I think the reasoning for leaving the column header visible is because, in a tree table, the first column includes the toggle (a checkbox if it's selectable) and the node/toggle/column text, so we don't move the column header down into the display for that cell like we do other cells in the row. Some screenshots: Desktop view Mobile view Mobile view in a page And for comparison, here is v5 since it didn't include the header Desktop view Mobile view |
bf3e532 to
0187fcc
Compare
0187fcc to
57eef9a
Compare
|
🎉 This PR is included in version 6.0.0-alpha.200 🎉 The release is available on: Your semantic-release bot 📦🚀 |
fixes #6865