Skip to content

Relayout editor during resize - #657

Open
DhruvShah-Dev wants to merge 4 commits into
TabularisDB:mainfrom
DhruvShah-Dev:fix/editor-resize-layout
Open

Relayout editor during resize#657
DhruvShah-Dev wants to merge 4 commits into
TabularisDB:mainfrom
DhruvShah-Dev:fix/editor-resize-layout

Conversation

@DhruvShah-Dev

Copy link
Copy Markdown
Contributor

Summary

  • Explicitly relayout the active Monaco editor while the SQL editor resize handle is dragged.
  • Relayout once more after releasing the handle so Monaco uses the final committed panel height.

Why

Issue #113 reports visual glitches while resizing the SQL editor. The resize handler updates the editor panel height directly during drag and commits React state on mouseup. Monaco's automatic layout can lag behind those manual DOM height writes, leaving stale measurements during the drag.

Fixes #113.

Validation

  • pnpm.cmd typecheck
  • pnpm.cmd lint

@DhruvShah-Dev
DhruvShah-Dev marked this pull request as ready for review August 17, 2026 18:58
@kilo-code-bot

kilo-code-bot Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • src/pages/Editor.tsx

Reviewed by glm-5.2 · Input: 28.6K · Output: 10K · Cached: 198.4K

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.

[Bug]: Display issue when resizing SQL editor

2 participants