Skip to content

add full-screen toggle for embedded editor pane#5

Merged
i1rr merged 1 commit into
masterfrom
claude/editor-fullscreen-toggle-2Pkky
Mar 16, 2026
Merged

add full-screen toggle for embedded editor pane#5
i1rr merged 1 commit into
masterfrom
claude/editor-fullscreen-toggle-2Pkky

Conversation

@i1rr
Copy link
Copy Markdown
Owner

@i1rr i1rr commented Mar 16, 2026

  • add ToggleEditorFullscreen action variant
  • add editor_fullscreen bool to DualPane; draw() gives 100% width to the active pane when true
  • clear editor_fullscreen on SwitchPane and close_editor_in_active
  • intercept Ctrl+E before EditorKeyInput routing in app map_key()
  • dispatch ToggleEditorFullscreen: toggle the flag when active pane is an editor, no-op otherwise
  • update draw_command_bar signature with editor_fullscreen param; show Ctrl+E:Fullscreen in normal editor mode and Ctrl+E:Exit fullscreen while fullscreen is active

https://claude.ai/code/session_011dpJMJEWrqscYNKheGcwj2

- add ToggleEditorFullscreen action variant
- add editor_fullscreen bool to DualPane; draw() gives 100% width to
  the active pane when true
- clear editor_fullscreen on SwitchPane and close_editor_in_active
- intercept Ctrl+E before EditorKeyInput routing in app map_key()
- dispatch ToggleEditorFullscreen: toggle the flag when active pane is
  an editor, no-op otherwise
- update draw_command_bar signature with editor_fullscreen param; show
  Ctrl+E:Fullscreen in normal editor mode and Ctrl+E:Exit fullscreen
  while fullscreen is active

https://claude.ai/code/session_011dpJMJEWrqscYNKheGcwj2
@i1rr i1rr merged commit 62b5dbf into master Mar 16, 2026
1 check passed
@i1rr i1rr deleted the claude/editor-fullscreen-toggle-2Pkky branch March 16, 2026 23:15
i1rr added a commit that referenced this pull request Apr 24, 2026
add full-screen toggle for embedded editor pane
i1rr added a commit that referenced this pull request Apr 24, 2026
add full-screen toggle for embedded editor pane
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