Skip to content

core/editor: commit cell edits when scrolling - #1206

Open
adamtoner wants to merge 1 commit into
glideapps:mainfrom
adamtoner:core/editor/close-on-scroll
Open

adamtoner wants to merge 1 commit into
glideapps:mainfrom
adamtoner:core/editor/close-on-scroll

Conversation

@adamtoner

Copy link
Copy Markdown

Editors retain their opening coordinates when the grid or page scrolls, leaving them detached from their cells. An editor can then appear to belong to a different row, making it unclear which record is being changed.

Commit and close the editor when the grid or a containing element scrolls, using the existing draft and validation handling to preserve the user's work. This deliberately ends the edit on scroll while allowing scrolling within the editor, its menus, or an unrelated grid. Related to #1159 and #1165.

Editors retain their opening coordinates when the grid or page scrolls,
leaving them detached from their cells. An editor can then appear to
belong to a different row, making it unclear which record is being changed.

Commit and close through the existing draft and validation handling to
preserve the user's work when scrolling moves the cell. This deliberately
ends the edit on scroll while allowing scrolling within the editor, its
menus, or an unrelated grid.

Related to glideapps#1159 and glideapps#1165.

This branch has not been deployed

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

1 participant