v5: Revisit table styles

[mdo]

#26678 has conflicts and tried to tackle a ton, so let's revisit style scoping, Sass maps, and more in v5.

[ysds]

A list of the table style issues:

• Nested table
  • Colored tables, prevent style leaks from nested tables & other table tweaks #26678, Table child elements (thead, tbody, tfoot, tr, td, th) should inherit parent background color #29244, Nested table inherits classes that cannot be reset #27800, prevents .table-striped to be automatically applied to inner tables #30251
• Color
  • Inconsistant .table-active color #24529 Inconsistent .table-active color caused by the opacity
  • .table-dark is used in two conflicting contexts #27879 .table-dark is used in two conflicting contexts
  • Suggestion for .table-hover #28160 Hover style for table rows/cells color classes
  • Table row-variants does not respect font-color #25675 (Closed) Table row-variants does not respect font-color
  • Contextual table classes apply tr class instead of td class to individual cells #29290 Contextual table classes apply tr class instead of td class to individual cells
  • Table child elements (thead, tbody, tfoot, tr, td, th) should inherit parent background color #29244 Table child elements (thead, tbody, tfoot, tr, td, th) should inherit parent background
• Spacing
  • Inconsistent component default/spacer padding #25057 Table cell padding in card
  • Wrong bottom margin in responsive tables #28214 (Closed) Table bottom margin

[mdo]

Not sure what to do about nested tables... we've only had a few requests over the last couple years for this after removing it in v4's development. Looking at the rest now...

[tmorehouse]

Perhaps a .table-reset class for child nested tables to negate the parent class inheritance?

[dimitor115]

@mdo Sorry but I don’t get it. Even if not many people requested a bug or problem there is still a reason to improve a selectors. Current behavior is just confusing. With not much effort we can provide a better tool. I can prepare the PR for example 😁

[MartijnCuppens]

#30466 closed this issue