Skip to content

Stop constraining grid cell editors to the viewport - #317

Merged
labkey-martyp merged 1 commit into
release26.7-SNAPSHOTfrom
26.7_fb_cell_editor_alignment
Sep 30, 2026
Merged

labkey-martyp merged 1 commit into
release26.7-SNAPSHOTfrom
26.7_fb_cell_editor_alignment

Conversation

@labkey-martyp

Copy link
Copy Markdown

Rationale

Tabbing past the right edge of the window in a wide EHR data-entry grid draws the cell editor over the wrong column and never scrolls the page, so the user types into a box that appears to belong to one column while the value goes to another. The LDK cell-editing plugin aligned editors with a trailing ?, which tells Ext to clamp the editor inside the browser viewport; EHR grids grow to fit their columns and rely on the page to scroll, so the clamp pinned off-screen editors to the window edge and left the browser nothing to scroll to. With the constraint removed the editor sits on its cell and focusing it scrolls the page, so tabbing beyond the width of the screen now scrolls horizontally to follow the active cell.

Related Pull Requests

None.

Changes

  • Drop the viewport constraint from the cell editor alignment in the LDK cell-editing plugin, so editors always align to their cell and the page scrolls to keep the active editor visible.

The trailing '?' on the editor alignment pinned editors for off-screen columns to the window edge instead of their cell, so tabbing past the right edge of a wide EHR grid drew the editor over the wrong column and never scrolled the page.
@labkey-martyp
labkey-martyp merged commit 077aec7 into release26.7-SNAPSHOT Sep 30, 2026
10 checks passed
@labkey-martyp
labkey-martyp deleted the 26.7_fb_cell_editor_alignment branch September 30, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants