LearnNewsExamplesServices
Frontmatter
id8982
titleFix Grid Horizontal Keyboard Scrolling
stateClosed
labels
bugairefactoring
assigneestobiu
createdAtFeb 4, 2026, 12:48 PM
updatedAtFeb 4, 2026, 12:51 PM
githubUrlhttps://github.com/neomjs/neo/issues/8982
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtFeb 4, 2026, 12:51 PM

Fix Grid Horizontal Keyboard Scrolling

Closed v12.0.0 bugairefactoring
tobiu
tobiu commented on Feb 4, 2026, 12:48 PM

Problem: Horizontal keyboard navigation (Left/Right arrows) in the Grid fails to scroll the container when reaching the edge of the visible area. Cause: Neo.grid.Container.scrollByColumns calls Neo.main.DomAccess.scrollTo using id: me.id. However, GridContainer overrides its root VDOM ID to me.getWrapperId() (suffix __wrapper). Therefore, the scrollTo call targets a non-existent DOM ID and fails silently. Fix: Update scrollByColumns to use id: me.vdom.id (or me.getWrapperId()).

tobiu added the bug label on Feb 4, 2026, 12:48 PM
tobiu added the ai label on Feb 4, 2026, 12:48 PM
tobiu added the refactoring label on Feb 4, 2026, 12:48 PM
tobiu referenced in commit 188e238 - "fix: Target correct DOM ID for Grid horizontal scrolling (#8982)" on Feb 4, 2026, 12:50 PM
tobiu
tobiu Feb 4, 2026, 12:51 PM

Input from Gemini 3 Pro:

✦ Fixed horizontal scrolling bug in GridContainer. scrollByColumns was targeting me.id instead of me.vdom.id (which carries the __wrapper suffix). Updated to use me.vdom.id to ensure DomAccess.scrollTo finds the correct scrollable element.

tobiu assigned to @tobiu on Feb 4, 2026, 12:51 PM
tobiu closed this issue on Feb 4, 2026, 12:51 PM