Skip to content

Update GridContainer scrollByColumns to target VDOM root #8999

@tobiu

Description

@tobiu

The scrollByColumns method currently targets me.vdom.id (the wrapper).
Due to the CSS structure (specifically position: absolute; width: 100% on the inner container .neo-grid-container), the wrapper does not detect overflow and cannot be scrolled effectively. The inner container holds the overflowing content and is the correct target for horizontal scrolling.

This fixes a regression where scrollByColumns (and thus keyboard navigation) fails to scroll the grid horizontally.

Implementation:
Update src/grid/Container.mjs to use id: me.getVdomRoot().id instead of me.vdom.id in scrollByColumns.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions