src/patternfly/components/Table/table.scss (1)

257-263: Sticky header/footer and tfoot styling look consistent; consider documenting .pf-m-sticky-footer alongside header

• The new sticky footer vars and block mirror the sticky header pattern and look correct: position: sticky + inset-block-end: 0, separate z-index vars, and a border pseudo-element to avoid double borders with the last tbody row.
• The switch to inset: 0 for the sticky header border pseudo-element is a nice cleanup and should be behavior‑preserving.
• The .pf-v6-c-table__tfoot block wiring (--table__tr--BorderBlockEndWidth: 0, vertical-align: top) matches the new table-tfoot.hbs usage.

One small follow-up suggestion: in the “Sticky table usage” docs table you still only list .pf-m-sticky-header. It would be clearer to add .pf-m-sticky-footer there as well, now that the modifier exists and is wired to table--HasStickyFooter.

Also applies to: 285-288, 328-349, 1303-1308

src/patternfly/components/Table/examples/Table.md (3)

3762-3927: Sticky header examples correctly migrated to table--HasStickyHeader; consider minor doc tweak

• The main sticky header example (table-sticky-header) and the nested-column sticky header example both now use table--HasStickyHeader=true, which matches the template wiring and SCSS.
• Extra body rows added to the sticky header example are just demo content and properly use table-td--data-label as before.

Optional doc improvement: in the “Sticky table usage” table you still only list .pf-m-sticky-header as a class. You might want to mention the table--HasStickyHeader flag there as the preferred way to enable it in examples, mirroring what you did for the plain variant.

Also applies to: 4371-4415

---

3933-4112: Sticky footer demo looks sound; document the modifier/flag more explicitly

The new “Sticky footer” example:

• Uses table--HasStickyFooter=true and a table-tfoot block, which is exactly what the new SCSS and table-tfoot.hbs expect.
• Uses <th scope="row"> in the footer row, which is good for accessibility.
• Has enough body rows to actually demonstrate the sticky behavior in the scrollable preview.

I’d suggest extending the “Sticky table usage” or “Table footer usage” docs to explicitly mention both .pf-m-sticky-footer and the table--HasStickyFooter flag, similar to how sticky headers and the plain variant are documented, so consumers know both the underlying class and the preferred flag.

---

5328-5445: Table footer examples and usage are consistent; fix the earlier ‘Applied to’ type to match

• The new “Table footer” example uses table-tfoot with a single summary row and <th scope="row">, which is semantically correct.
• The usage table documents .pf-v6-c-table__tfoot applied to tfoot, matching the new table-tfoot.hbs template and SCSS.

One follow-up: in the earlier “Basic table usage” table, .pf-v6-c-table__thead, .pf-v6-c-table__tbody, and .pf-v6-c-table__tfoot are documented as applied to <tr>, but the implementation (and these examples) apply them to <thead>, <tbody>, and <tfoot>. It would be good to update that “Applied to” column for consistency.

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro