Skip to content

Phase-1 follow-up: Avalonia interlinear editor (Words Analyses)#965

Draft
johnml1135 wants to merge 1 commit into
phase1-basefrom
phase1-followup-interlinear
Draft

Phase-1 follow-up: Avalonia interlinear editor (Words Analyses)#965
johnml1135 wants to merge 1 commit into
phase1-basefrom
phase1-followup-interlinear

Conversation

@johnml1135

@johnml1135 johnml1135 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Phase-1 follow-up: Avalonia interlinear editor (Words → Analyses)

Stacked on #964 (base). Restores and activates the native Avalonia interlinear editor for the Words → Analyses morph-bundle detail pane.

  • Adds the 11 interlinear files (region editor, analysis model, projector/write-back, InterlinearSlicePlugin, tests).
  • Restores the plugin registration in RegionEditorPlugins and the burn-down census entry.
  • Flip: registers "Analyses" in LexicalEditFeatureCatalog (with new display-name/description strings), so it flows into the catalog-sourced DefaultSupportedTools and the editor resolves to Avalonia under UIMode=New (default still Legacy); removed from Phase1FollowUpSurfaceTools.

Note: since this branch was rebased onto the squashed base PR, "flip" now means a catalog entry rather than a hardcoded array edit — as a side effect, this also adds a new "Words Analyses" row to the Tools→Options "Manage Individual Features" dialog (under "Other record types"), visible under UIMode=New.

Verification: build green; census + RegisteredRecordEditTools("Analyses") + interlinear suites all passing (FwAvaloniaTests 846/846, xWorksTests 1664/1666, 2 pre-existing skips). Review after the base PR.


This change is Reviewable

@github-actions

This comment has been minimized.

@johnml1135 johnml1135 force-pushed the phase1-followup-interlinear branch from 17bd535 to 5120ef6 Compare June 25, 2026 02:56
@johnml1135 johnml1135 force-pushed the phase1-base branch 2 times, most recently from 3a905c6 to b423a19 Compare June 25, 2026 10:38
@johnml1135 johnml1135 force-pushed the phase1-followup-interlinear branch from 5120ef6 to 3c5893e Compare June 25, 2026 10:38
@johnml1135 johnml1135 force-pushed the phase1-base branch 3 times, most recently from e9cdaa0 to c37eb02 Compare July 2, 2026 17:38
johnml1135 added a commit that referenced this pull request Jul 3, 2026
Verified the stacked follow-up PRs this base PR's description promises
actually exist and are correctly chained: phase1-followup-interlinear
(#965) -> phase1-followup-rule (#966) -> phase1-followup-table (#967),
all open/draft. The gating comments explaining why the browse table and
ChorusNotesPlugin are inert said "a follow-up PR" generically -- name
phase1-followup-table specifically (matching the sibling comment in
LexicalEditSurfaceRegistry.cs, which already names avalonia-interlinear-
editor/avalonia-rule-formula-editor per tool).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@johnml1135 johnml1135 force-pushed the phase1-followup-interlinear branch 2 times, most recently from d6003e4 to c937d8b Compare July 4, 2026 14:32
Restore the interlinear surface on top of phase1-base and activate it:
- add back the 11 interlinear files (InterlinearRegionEditor + analysis
  model + projector/write-back + plugin + their tests, incl. the
  FwAvalonia model and Visual tests)
- restore the InterlinearSlicePlugin registration in RegionEditorPlugins
- restore the interlinear class name + resolve assertion in the
  burn-down census
- FLIP: register "Analyses" in LexicalEditFeatureCatalog (with new
  display-name/description strings), so it flows into the catalog-
  sourced DefaultSupportedTools and the Words Analyses detail editor
  resolves to Avalonia under UIMode=New; removed from
  Phase1FollowUpSurfaceTools
- add the "Analyses" TestCase back to
  RegisteredRecordEditTools_ResolveToAvalonia

The browse "Analyses" list pane stays inert (table follow-up territory).

Rebased onto the squashed phase1-base (post PR #964 review): the base PR
had since refactored tool registration to be catalog-driven, so this
flip now also adds a "Words Analyses" row to the Tools->Options "Manage
Individual Features" dialog under UIMode=New.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@johnml1135 johnml1135 force-pushed the phase1-followup-interlinear branch from c937d8b to bebe347 Compare July 4, 2026 15:26
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