Skip to content

Develop - #1

Merged
JDonaghy merged 12 commits into
mainfrom
develop
Feb 15, 2026
Merged

Develop#1
JDonaghy merged 12 commits into
mainfrom
develop

Conversation

@JDonaghy

Copy link
Copy Markdown
Owner

No description provided.

JDonaghy and others added 12 commits February 13, 2026 18:39
Implements basic . repeat for insert/delete operations.
Supports count prefix (3.). 4 tests added.
Add VSCode-style keybindings and focus management for file explorer:

- Ctrl-Shift-E to focus explorer, Escape to return to editor
- Active file highlighting in tree with auto-expand parents
- Auto-focus editor after opening files
- Disable TreeView search to prevent keyboard interference
- Comprehensive error handling and filename validation
- Focus management fixes for seamless editing workflow

Technical: Use Rc<RefCell<>> for widget refs in Relm4, allow deprecated TreeView (GTK 4.10+)

Tests: 232 passing, clippy clean
Single-click in the file explorer opens files as reusable preview tabs
(italic/dimmed) that get replaced when clicking another file. Double-click
opens permanently. Editing or saving a preview auto-promotes it to permanent.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- PLAN.md: 38 lines (was 545) - shows Phase 4 completion
- PROJECT_STATE.md: 100 lines (was 443) - condensed status
- README.md: 136 lines (was 184) - streamlined
- Archive original Phase 4 plan

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Tab bar always visible (even with single tab)
- Double-click files opens in new tab (not current window)
- Undo clears dirty flag when back to saved state
- Tab clicking switches active tab
- Archive Phase 4 plan (preview mode complete)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@JDonaghy
JDonaghy merged commit 150d588 into main Feb 15, 2026
1 check failed
JDonaghy added a commit that referenced this pull request Apr 29, 2026
Lift the bespoke GTK completion popup paint at
`src/gtk/draw::draw_completion_popup` into
`quadraui::gtk::draw_completions`, mirroring the TUI rasteriser
shipped in #266. Both backends now paint completions through
`quadraui::Completions` — closes the smallest remaining duplication
slot from PLAN.md "🎯 NEXT FOCUS" item #1 after Phase C Stage 1
(#276).

Verbatim port: full bg fill, full 4-side border (intrinsic
divergence vs TUI's side-only `│` borders), per-item selected-row
highlight, candidate label as `" {label}"` via Pango. The host
builds `quadraui::Completions` via the existing
`render::completion_menu_to_quadraui_completions` adapter, computes
placement via `Completions::layout(...)`, then delegates to the
lifted rasteriser through the `quadraui_gtk::draw_completions`
shim. The wrapper preserves its `Option<(f64,f64,f64,f64)>` return
shape so the existing `completion_popup_rect_out` modal-stack
click integration keeps working unchanged.

Net: +99 LOC of shared paint in quadraui; vimcode-private paint
approximately neutral. Quality gate clean: lib + integration tests
pass on `--no-default-features`; GTK build + clippy clean.

Closes #285

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
JDonaghy added a commit that referenced this pull request May 1, 2026
PROJECT_STATE.md gains a Session 346 entry capturing:
- 5 PRs that landed (cell_quantum #297, TUI MSV harness #298, TUI
  TreeView harness #299, quadraui extraction #300, plus 3 doc commits
  direct to develop)
- Quadraui now lives at its own repo; vimcode consumes via path-dep
  sibling
- Cross-repo blocked-label tracking now in place (vimcode #296, #282,
  #301, #302 all blocked on quadraui issues #1-#7 as appropriate)
- Migration prerequisites rule means no further consumer migrations
  land in vimcode until quadraui's rasterisers + harnesses ship

PLAN.md "Course correction" section: 7 steps now show ✅ status with
PR/SHA references for steps 1-5 (shipped) and ⏳ with cross-repo issue
links for steps 6-7 (in quadraui repo). New "Cross-repo prereq
tracking" subsection explains how the blocked label flows through
plan-next.

Doc-only; landing direct to develop.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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