Skip to content

DataGrid - Focused cell highlight on sticky columns is still visible even when the focused cell is outside the viewport (T1310313)#33284

Open
markallenramirez wants to merge 3 commits intoDevExpress:26_1from
markallenramirez:bug_T1310313/26_1
Open

DataGrid - Focused cell highlight on sticky columns is still visible even when the focused cell is outside the viewport (T1310313)#33284
markallenramirez wants to merge 3 commits intoDevExpress:26_1from
markallenramirez:bug_T1310313/26_1

Conversation

@markallenramirez
Copy link
Copy Markdown
Contributor

No description provided.

@markallenramirez markallenramirez self-assigned this Apr 15, 2026
@markallenramirez markallenramirez requested a review from a team as a code owner April 15, 2026 11:12
Copilot AI review requested due to automatic review settings April 15, 2026 11:12
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 a DataGrid sticky-columns focus-overlay behavior where the focused cell highlight could remain visible on sticky columns even after the focused cell scrolls out of the viewport.

Changes:

  • Removed the sticky-columns-specific getFocusOverlayContainer override so the focus overlay uses the base EditorFactory container logic.

Copilot AI review requested due to automatic review settings April 17, 2026 08:27
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread e2e/testcafe-devextreme/tests/dataGrid/sticky/common/withEditing.ts Outdated
Copilot AI review requested due to automatic review settings April 17, 2026 09:48
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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants