LearnNewsExamplesServices
Frontmatter
id18951
titleThe grid cell-editing fixture pushes both grids 31 px off-screen
stateClosed
labels
bugaitestinggrid
assigneesneo-opus-ada
createdAtSep 19, 2026, 2:32 PM
updatedAtSep 19, 2026, 2:55 PM
githubUrlhttps://github.com/neomjs/neo/issues/18951
authorneo-opus-ada
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
contentTrust
projected
quarantined0
signals[]
blockedBy[]
blocking[]
closedAtSep 19, 2026, 2:55 PM

The grid cell-editing fixture pushes both grids 31 px off-screen

neo-opus-ada
neo-opus-ada commented on Sep 19, 2026, 2:32 PM

Context

Reported by @tobiu from the fixture page itself (test/playwright/component/apps/grid-cell-editing/index.html): an "info text" sits clipped at the left edge of the grids. Measured at 1400×900 on dev@ab2e7cd17c, the defect is larger than the label.

The Problem

The fixture's <button id="grid-cell-editing-outside">Outside the grids</button> — the focus target outside the app that the reprojection and click-away arms click — is a sibling of the Neo Viewport in <body>. The theme makes the body a centered flex row (display: flex; flex-direction: row; align-items: center), and the Viewport is 100% wide. So the row holds 62 px + 1400 px in a 1400 px window, and centering splits the overflow:

element measured
button x = −31, 62 px wide: half off-screen, its label wrapped and clipped
Viewport and both grids x = 31, 1400 px wide: the rightmost 31 px of both grids — the locked-end Note and c39 columns — are off-screen

Every arm on this fixture has run with the locked-end body partly outside the window.

The Fix

Give the focus target its own strip above the app in index.html: the body stacks as a column, the button keeps its natural size at the top left, and the Viewport fills the rest at full width. Prototyped via an injected style on the same head: the button renders at x = 4 in full, and both grids span x = 0 to 1400. The pooled grid renders 18 rows instead of 19 in the remaining 871 px.

Acceptance Criteria

  • At 1400×900, the button renders in full inside the window, and both grids start at x = 0 and end at the window's right edge.
  • The button stays outside the Neo app (a plain element), so its role as the non-Neo focus target is unchanged.
  • Every spec on this fixture passes: component grid, e2e grid/CellEditing*. The 19 → 18 row change is checked against any arm that counts rows.

Out of Scope

  • The fixture's grids and data (app.mjs).
  • Any other test app's layout.

Related

#18762 (the epic whose arms run on this fixture) · #18927 (added the outside-focus arms)

Live latest-open sweep: checked the latest 20 open issues at 2026-09-19T12:40Z; searches for grid-cell-editing fixture, outside the grids, fixture viewport overflow found nothing equivalent.

Origin Session ID: 6ecb7b5f-dc26-48a3-8e49-7232159377c1