Skip to content

DataGrid: fix width calculation when column hiding enabled (T1326188)#33241

Open
anna-shakhova wants to merge 1 commit intoDevExpress:26_1from
anna-shakhova:26_1__T1326460_scroll_column_hide
Open

DataGrid: fix width calculation when column hiding enabled (T1326188)#33241
anna-shakhova wants to merge 1 commit intoDevExpress:26_1from
anna-shakhova:26_1__T1326460_scroll_column_hide

Conversation

@anna-shakhova
Copy link
Copy Markdown
Contributor

No description provided.

@anna-shakhova anna-shakhova self-assigned this Apr 13, 2026
Copilot AI review requested due to automatic review settings April 13, 2026 13:37
@anna-shakhova anna-shakhova requested a review from a team as a code owner April 13, 2026 13:37
Copy link
Copy Markdown
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 width calculations in adaptivity mode to prevent unwanted horizontal scrolling when column hiding is enabled and a vertical scrollbar is present.

Changes:

  • Adjusted redundant-column hiding logic to use “available width” that accounts for the vertical scrollbar width.
  • Tightened TypeScript typing around the hiding-columns queue and the hideRedundantColumns method.
  • Added a TestCafe regression test to ensure no horizontal scrollbar appears in the reported scenario.

Reviewed changes

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

File Description
packages/devextreme/js/__internal/grids/grid_core/adaptivity/m_adaptivity.ts Updates adaptivity width calculations to subtract vertical scrollbar width when deciding whether columns fit.
e2e/testcafe-devextreme/tests/dataGrid/common/scrolling.ts Adds an e2e regression test validating no horizontal overflow with columnHidingEnabled + vertical scrollbar.

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.

3 participants