LearnNewsExamplesServices
Frontmatter
id9634
titleGrid Container: Remove redundent grid wrapper node
stateClosed
labels
enhancementai
assigneestobiu
createdAtApr 2, 2026, 3:55 PM
updatedAtApr 2, 2026, 4:06 PM
githubUrlhttps://github.com/neomjs/neo/issues/9634
authortobiu
commentsCount1
parentIssue9626
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtApr 2, 2026, 4:06 PM

Grid Container: Remove redundent grid wrapper node

Closedenhancementai
tobiu
tobiu commented on Apr 2, 2026, 3:55 PM

This is a sub-epic for #9626.

As part of the Unified Scroll Architecture Redesign, we need to rip out the unnecessary .neo-grid-wrapper node surrounding the Grid Container.

Tasks:

  • Change _vdom in GridContainer to directly return a single root node.
  • Remove redundant overriden methods: getVdomRoot, getVnodeRoot, and getWrapperId.
  • Remove wrapper assignments during Grid initialization (me.vdom.id = me.getWrapperId()).
  • Flatten Container.scss to merge wrapper styles directly into .neo-grid-container.
  • Ensure all E2E tests are compatible with the flattened DOM tree.
tobiu added the enhancement label on Apr 2, 2026, 3:55 PM
tobiu added the ai label on Apr 2, 2026, 3:55 PM
tobiu added parent issue #9626 on Apr 2, 2026, 3:56 PM
tobiu referenced in commit cf31306 - "feat: remove redundent grid wrapper node (#9634)" on Apr 2, 2026, 4:05 PM
tobiu
tobiu Apr 2, 2026, 4:05 PM

Input from Gemini 3.1 Pro (Antigravity):

✦ The wrapper node removal for GridContainer is complete!

  • Dropped getVdomRoot, getVnodeRoot, and getWrapperId structurally to align with .neo-grid-container flattened Base handling.
  • Hoisted layout parameters out of Container.scss .neo-grid-wrapper directly into .neo-grid-container.
  • Updated SortZone.mjs config array to no longer mock the removed wrapper.

Code has been pushed and verified on local DevIndex testing.

tobiu assigned to @tobiu on Apr 2, 2026, 4:06 PM
tobiu closed this issue on Apr 2, 2026, 4:06 PM