Skip to content

fix(data-table): sync pinned column hover transition#399

Merged
IzumiSy merged 2 commits into
mainfrom
fix/data-table-pinned-hover-transition
Jul 23, 2026
Merged

fix(data-table): sync pinned column hover transition#399
IzumiSy merged 2 commits into
mainfrom
fix/data-table-pinned-hover-transition

Conversation

@IzumiSy

@IzumiSy IzumiSy commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Motivation

Pinned DataTable body cells use their own opaque background so horizontally scrolled content does not bleed through sticky columns. Their hover and selected colors were updated at the cell level with group-hover, while the rest of the row relied on the row-level transition-colors style.

That mismatch made pinned columns snap to the hover color slightly earlier than scrollable columns, which was noticeable in the DataTable lab and made the row feel visually inconsistent.

This behavior belongs to the in-flight pinned-column work and is already covered by that unreleased feature's existing changeset.

Design Decision

Chosen approach

Add transition-colors to pinned body cells so their background color animates with the same timing as the rest of the row.

Alternatives considered

Changing the row hover implementation would have touched all table cells. This fix stays local to the sticky-cell path, which is the only place the timing diverged.

Summary

  • add transition-colors to pinned DataTable body cells
  • keep the existing sticky background and hover color logic intact
  • rely on the existing unreleased pinned-column changeset instead of adding a new one

@IzumiSy IzumiSy self-assigned this Jul 23, 2026
@IzumiSy
IzumiSy marked this pull request as ready for review July 23, 2026 07:06
@IzumiSy
IzumiSy requested a review from a team as a code owner July 23, 2026 07:06
@IzumiSy
IzumiSy merged commit c99041f into main Jul 23, 2026
4 of 5 checks passed
@IzumiSy
IzumiSy deleted the fix/data-table-pinned-hover-transition branch July 23, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants