From 297814967951f277518bfb07ff8868edec09a0fc Mon Sep 17 00:00:00 2001 From: lucaronin Date: Sat, 4 Apr 2026 14:45:02 +0200 Subject: [PATCH] fix: rename "Link existing" to "Add relationship" in Inspector Aligns relationship button label with the "Add property" pattern used elsewhere in the Inspector panel. Co-Authored-By: Claude Opus 4.6 (1M context) --- src/components/InspectorPanels.test.tsx | 16 ++++++++-------- src/components/inspector/RelationshipsPanel.tsx | 4 ++-- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/components/InspectorPanels.test.tsx b/src/components/InspectorPanels.test.tsx index c4402bde..301441e6 100644 --- a/src/components/InspectorPanels.test.tsx +++ b/src/components/InspectorPanels.test.tsx @@ -139,7 +139,7 @@ describe('DynamicRelationshipsPanel', () => { expect(screen.getByText('Luca')).toBeInTheDocument() }) - it('renders + Link existing button when onAddProperty provided', () => { + it('renders + Add relationship button when onAddProperty provided', () => { render( { onAddProperty={onAddProperty} /> ) - expect(screen.getByText('+ Link existing')).toBeInTheDocument() + expect(screen.getByText('+ Add relationship')).toBeInTheDocument() }) it('opens add relationship form when button clicked', () => { @@ -162,7 +162,7 @@ describe('DynamicRelationshipsPanel', () => { onAddProperty={onAddProperty} /> ) - fireEvent.click(screen.getByText('+ Link existing')) + fireEvent.click(screen.getByText('+ Add relationship')) expect(screen.getByPlaceholderText('Relationship name')).toBeInTheDocument() expect(screen.getByPlaceholderText('Note title')).toBeInTheDocument() }) @@ -177,7 +177,7 @@ describe('DynamicRelationshipsPanel', () => { onAddProperty={onAddProperty} /> ) - fireEvent.click(screen.getByText('+ Link existing')) + fireEvent.click(screen.getByText('+ Add relationship')) fireEvent.change(screen.getByPlaceholderText('Relationship name'), { target: { value: 'Related to' } }) fireEvent.change(screen.getByPlaceholderText('Note title'), { target: { value: 'AI' } }) fireEvent.click(screen.getByText('Add')) @@ -194,9 +194,9 @@ describe('DynamicRelationshipsPanel', () => { onAddProperty={onAddProperty} /> ) - fireEvent.click(screen.getByText('+ Link existing')) + fireEvent.click(screen.getByText('+ Add relationship')) fireEvent.click(screen.getByText('Cancel')) - expect(screen.getByText('+ Link existing')).toBeInTheDocument() + expect(screen.getByText('+ Add relationship')).toBeInTheDocument() }) it('dims archived entries', () => { @@ -591,7 +591,7 @@ describe('DynamicRelationshipsPanel', () => { onCreateAndOpenNote={onCreateAndOpenNote} /> ) - fireEvent.click(screen.getByText('+ Link existing')) + fireEvent.click(screen.getByText('+ Add relationship')) fireEvent.change(screen.getByPlaceholderText('Relationship name'), { target: { value: 'Mentions' } }) const noteInput = screen.getByPlaceholderText('Note title') fireEvent.focus(noteInput) @@ -610,7 +610,7 @@ describe('DynamicRelationshipsPanel', () => { onCreateAndOpenNote={onCreateAndOpenNote} /> ) - fireEvent.click(screen.getByText('+ Link existing')) + fireEvent.click(screen.getByText('+ Add relationship')) fireEvent.change(screen.getByPlaceholderText('Relationship name'), { target: { value: 'Mentions' } }) const noteInput = screen.getByPlaceholderText('Note title') fireEvent.focus(noteInput) diff --git a/src/components/inspector/RelationshipsPanel.tsx b/src/components/inspector/RelationshipsPanel.tsx index 6567258e..a2e49c1e 100644 --- a/src/components/inspector/RelationshipsPanel.tsx +++ b/src/components/inspector/RelationshipsPanel.tsx @@ -342,7 +342,7 @@ function AddRelationshipForm({ entries, onAddProperty, onCreateAndOpenNote }: { if (!showForm) { return ( - + ) } @@ -390,7 +390,7 @@ function updateRefsForAddition(refs: string[], noteTitle: string): FrontmatterVa function DisabledLinkButton() { return ( - + ) }