LearnNewsExamplesServices
Frontmatter
id8999
titleUpdate GridContainer scrollByColumns to target VDOM root
stateClosed
labels
bugairegression
assigneestobiu
createdAtFeb 5, 2026, 11:44 AM
updatedAtFeb 5, 2026, 11:46 AM
githubUrlhttps://github.com/neomjs/neo/issues/8999
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtFeb 5, 2026, 11:46 AM

Update GridContainer scrollByColumns to target VDOM root

Closed v12.0.0 bugairegression
tobiu
tobiu commented on Feb 5, 2026, 11:44 AM

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.

tobiu added the bug label on Feb 5, 2026, 11:44 AM
tobiu added the ai label on Feb 5, 2026, 11:44 AM
tobiu added the regression label on Feb 5, 2026, 11:44 AM
tobiu assigned to @tobiu on Feb 5, 2026, 11:45 AM
tobiu referenced in commit 23b0979 - "fix: Update GridContainer scrollByColumns to target VDOM root (#8999)" on Feb 5, 2026, 11:45 AM
tobiu
tobiu Feb 5, 2026, 11:46 AM

Input from Gemini 3 Pro:

✦ I have applied the fix to target me.getVdomRoot().id instead of me.vdom.id. This ensures scrollByColumns functions correctly with the updated DOM structure where the inner container is the effective scroll target.

tobiu closed this issue on Feb 5, 2026, 11:46 AM