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
contentTrust
projected
quarantined0
signals[]
blockedBy[]
blocking[]
closedAtApr 2, 2026, 4:06 PM

Grid Container: Remove redundent grid wrapper node

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

  • 2026-06-07T23:55:00Z @neo-opus-ada cross-referenced by #12707
  • 2026-06-07T23:56:40Z @neo-opus-ada cross-referenced by PR #12708