chore(table): table selectors

[mattnolting]

closes #5549

What this PR does

• Adds .#{$table}__tr, .#{$table}__th, and .#{$table}__td selectors.
• Adds direct child ex: .#{$table} > thead combinator to elements without an applied class (tbody, thead, *)
• Adds a :where() :pseudo to retain specificity while adding versioned class to tr, th, and td

tbody and thead are ignored in this class update in that they can be targeted with a direct child pseudo. tr is updated as adding a I saw tr:where(.#{$table}__tr to be preferable over :where(.#{$table}) > tbody > tr

[patternfly-build]

Preview: https://patternfly-pr-5558.surge.sh

A11y report: https://patternfly-pr-5558-a11y.surge.sh

[mattnolting]

@mcoker @srambach Wdyt about this approach to table style versioning without changing specificity?

What this PR does

• Adds .#{$table}__tr, .#{$table}__th, and .#{$table}__td selectors.
• Adds direct child ex: .#{$table} > thead combinator to elements without an applied class (tbody, thead, *)
• Adds a :where() :pseudo to retain specificity while adding versioned class to tr, th, and td

[srambach]

Looks good afaict 👍

[mcoker]

Lookin good! Lots of little bugs, most have created visual discrepancies (how I found them), but all CSS and can be cleaned up later except for the table grid one. Depending on what you want to do there, that might require markup changes. Lemme know if I can help.

Also... can we still run screenshot tests? I think I found all the issues, but this would be a great case for screenshot comparisons to find issues.

And this one is not tied to a prefixed class, but I didn't see the change in the changeset, so leaving it here -

patternfly/src/patternfly/components/Table/table.scss

Line 407 in 01815c9

tr.pf-m-striped { // tr

should probably be

tr:where(.#{$table}__tr).pf-m-striped { // tr

[mcoker]

🥳

[patternfly-build]

🎉 This PR is included in version 5.0.0-alpha.55 🎉

The release is available on:

• GitHub release
• npm package (@alpha dist-tag)

Your semantic-release bot 📦🚀