Skip to content

Phase-1 follow-up: activate the Avalonia browse table#967

Draft
johnml1135 wants to merge 1 commit into
phase1-followup-rulefrom
phase1-followup-table
Draft

Phase-1 follow-up: activate the Avalonia browse table#967
johnml1135 wants to merge 1 commit into
phase1-followup-rulefrom
phase1-followup-table

Conversation

@johnml1135

@johnml1135 johnml1135 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Phase-1 follow-up: activate the Avalonia browse table

Stacked on the rule-formula follow-up. Pure activation flip — the table view code (LexicalBrowseView, LexicalBrowseHostControl, BulkEditBarView, ClerkBrowse*) and its RecordBrowseView wiring already shipped (dormant) in the base PR, because they are fused to base wiring and could not be carved into a separate code PR.

  • Flip: moves the 8 browse tool names (lexiconEdit, lexiconBrowse, notebookEdit, notebookBrowse, Analyses, toolBulkEditWordforms, featureTypesAdvancedEdit, reversalToolReversalIndexPOS) from Phase1FollowUpBrowseTools into SupportedAvaloniaBrowseToolNames.
  • Flips the 4 base inert-browse resolver tests back to asserting Avalonia under New.

2-file change. Verification: build green; resolver/browse suites 142/142.


This change is Reviewable

Stacked on the rule-formula follow-up. The browse table view code
(LexicalBrowseView, LexicalBrowseHostControl, BulkEditBarView, ClerkBrowse*)
and its RecordBrowseView product wiring already shipped dormant in the base PR
(they are fused to base wiring, so they could not be carved out). This PR is
the activation flip:
- move the 8 browse tool names (lexiconEdit, lexiconBrowse, notebookEdit,
  notebookBrowse, Analyses, toolBulkEditWordforms, featureTypesAdvancedEdit,
  reversalToolReversalIndexPOS) from Phase1FollowUpBrowseTools into
  SupportedAvaloniaBrowseToolNames; Phase1FollowUpBrowseTools is now empty
- flip the 4 base inert-browse resolver tests back to asserting Avalonia under
  New (LexiconBrowse, LexiconEdit, RegisteredNonLexiconTool x6, ExplicitOverride),
  restoring the pre-split _YieldsAvalonia expectations

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

⚠️ Commit Message Format Issues ⚠️

commit 47dbc776c5:
13: B1 Line exceeds max length (82>80): " New (LexiconBrowse, LexiconEdit, RegisteredNonLexiconTool x6, ExplicitOverride),"

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