Skip to content

T1321834 - DataGrid – Incorrect tooltip on the column header if it too narrow and has sort index#32571

Open
dmirgaev wants to merge 4 commits intoDevExpress:26_1from
dmirgaev:26_1__T1321834_incorrect_tooltip
Open

T1321834 - DataGrid – Incorrect tooltip on the column header if it too narrow and has sort index#32571
dmirgaev wants to merge 4 commits intoDevExpress:26_1from
dmirgaev:26_1__T1321834_incorrect_tooltip

Conversation

@dmirgaev
Copy link
Contributor

No description provided.

@dmirgaev dmirgaev self-assigned this Feb 13, 2026
@dmirgaev dmirgaev requested a review from a team as a code owner February 13, 2026 13:17
Copilot AI review requested due to automatic review settings February 13, 2026 13:17
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes DataGrid header-cell hint text so that, when sort indexes are shown and the header is narrow, the tooltip displays the column caption (instead of the sort index text from the hovered indicator element).

Changes:

  • Update cell-hint tooltip text selection for header rows to prefer column.caption over the hovered element’s text.
  • Add Jest coverage for cellHintEnabled tooltip behavior (header sort index indicator vs. non-header cells).
  • Extend the header cell test model to expose the sort-index icon element.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
packages/devextreme/js/__internal/grids/grid_core/views/m_columns_view.ts Adjusts tooltip text logic to use the column caption for header rows.
packages/devextreme/js/__internal/grids/grid_core/column_headers/__tests__/m_column_headers.test.ts Adds tests ensuring tooltip shows caption for sort index indicator and cell text for data cells.
packages/devextreme/js/__internal/grids/grid_core/__tests__/__mock__/model/cell/header_cell.ts Adds a helper to locate the sort-index icon within a header cell for tests.

@dmirgaev dmirgaev force-pushed the 26_1__T1321834_incorrect_tooltip branch from 35f8412 to c043c91 Compare February 13, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants