DataGrid: fix Tab press handler in filter range (T1312521)

[anna-shakhova]

No description provided.

[Copilot]

Pull request overview

This PR refactors the DataGrid filter row code by removing unnecessary/redundant code and improving type safety and test organization.

Changes:

• Refactored filter range overlay container positioning to be relative to grid headers instead of viewport
• Improved Tab key navigation logic to handle the last column edge case
• Enhanced type safety by adding MenuInternal type import and removing TypeScript error suppressions
• Reorganized TestCafe tests by separating visual and functional tests into dedicated files

Reviewed changes

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

Show a summary per file

File	Description
packages/devextreme/testing/tests/DevExpress.ui.widgets.dataGrid/filterRow.tests.js	Added helper function getFilterRowOverlay() to simplify test assertions and updated all overlay selector references
packages/devextreme/js/__internal/grids/grid_core/filter/m_filter_row.ts	Changed overlay container from viewport to grid element, improved Tab navigation to handle last column, added MenuInternal type, removed unnecessary blur trigger
e2e/testcafe-devextreme/tests/dataGrid/common/filterRow/visual.ts	New file containing visual/screenshot tests previously in functional.ts
e2e/testcafe-devextreme/tests/dataGrid/common/filterRow/functional.ts	Removed visual tests (moved to visual.ts), cleaned up imports, reorganized test order
e2e/testcafe-devextreme/tests/dataGrid/common/filterRow/T1163100_changeFIlterIcon.visual_matrix.ts	New test file for T1163100 filter icon change visual tests

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (2)

e2e/testcafe-devextreme/tests/dataGrid/common/filterRow/functional.ts:170

• The test uses test.only which will cause other tests in this file to be skipped during test execution. This should be changed to test before merging to ensure all tests run in CI.
  e2e/testcafe-devextreme/tests/dataGrid/common/filterRow/functional.ts:205
• The test uses test.only which will cause other tests in this file to be skipped during test execution. This should be changed to test before merging to ensure all tests run in CI.

[Copilot]

Pull request overview

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