Skip to content

[WC-349]: Prevent scroll on column selector focus - #1821

Merged
rahmanunver merged 2 commits into
mainfrom
fix/wc-3049_column_selector_scroll
Sep 8, 2025
Merged

[WC-349]: Prevent scroll on column selector focus#1821
rahmanunver merged 2 commits into
mainfrom
fix/wc-3049_column_selector_scroll

Conversation

@rahmanunver

Copy link
Copy Markdown
Contributor

Pull request type

Bug fix (non-breaking change which fixes an issue)


Description

The referenced column selector element was being focused after loading DG2 as a solution for focus behavior, it was causing a horizontal scroll issue on initial load. We are keeping the focus behavior same but preventing the scroll now.

What should be covered while testing?

A dg2 nested in container with column selector enabled. See the test project.

@rahmanunver
rahmanunver requested a review from a team as a code owner August 11, 2025 17:40
Comment thread packages/pluggableWidgets/datagrid-web/src/components/ColumnSelector.tsx Outdated
@rahmanunver
rahmanunver force-pushed the fix/wc-3049_column_selector_scroll branch from 7905fbc to c71b2ea Compare August 12, 2025 13:31
@gjulivan

Copy link
Copy Markdown
Collaborator

need rebase @rahmanunver

@r0b1n
r0b1n force-pushed the fix/wc-3049_column_selector_scroll branch from c71b2ea to eaaced5 Compare August 27, 2025 08:52
@rahmanunver
rahmanunver force-pushed the fix/wc-3049_column_selector_scroll branch from eaaced5 to 3cb44e0 Compare August 28, 2025 07:52
leonardomendix
leonardomendix previously approved these changes Sep 4, 2025
@gjulivan
gjulivan force-pushed the fix/wc-3049_column_selector_scroll branch from 2e95703 to 31a6fc7 Compare September 4, 2025 19:15
leonardomendix
leonardomendix previously approved these changes Sep 5, 2025
@rahmanunver
rahmanunver merged commit 9e6165a into main Sep 8, 2025
14 checks passed
@rahmanunver
rahmanunver deleted the fix/wc-3049_column_selector_scroll branch September 8, 2025 11:43
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.

4 participants