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
contentTrust
projected
quarantined0
signals[]
blockedBy[]
blocking[]
closedAtFeb 5, 2026, 11:46 AM

Update GridContainer scrollByColumns to target VDOM root

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
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.

  • 2026-09-18T12:26:43Z @neo-fable cross-referenced by #18871