davidl/tolaria
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: rename "Link existing" to "Add relationship" in Inspector

Browse Source 
Aligns relationship button label with the "Add property" pattern used
elsewhere in the Inspector panel.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Test
2026-04-04 14:45:02 +02:00
parent 2feff35764
commit 1d3927ca2b
2 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/components/InspectorPanels.test.tsx
View File

@@ -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(
<DynamicRelationshipsPanel
typeEntryMap={{}}
@@ -149,7 +149,7 @@ describe('DynamicRelationshipsPanel', () => {
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)

4
src/components/inspector/RelationshipsPanel.tsx
View File

@@ -342,7 +342,7 @@ function AddRelationshipForm({ entries, onAddProperty, onCreateAndOpenNote }: {
if (!showForm) {
return (
<button className="mt-2 w-full border border-border bg-transparent text-center text-muted-foreground" style={{ borderRadius: 6, padding: '6px 12px', fontSize: 12, cursor: 'pointer' }} onClick={() => { setShowForm(true); setTimeout(() => keyInputRef.current?.focus(), 0) }}>+ Link existing</button>
<button className="mt-2 w-full border border-border bg-transparent text-center text-muted-foreground" style={{ borderRadius: 6, padding: '6px 12px', fontSize: 12, cursor: 'pointer' }} onClick={() => { setShowForm(true); setTimeout(() => keyInputRef.current?.focus(), 0) }}>+ Add relationship</button>
)
}
@@ -390,7 +390,7 @@ function updateRefsForAddition(refs: string[], noteTitle: string): FrontmatterVa
function DisabledLinkButton() {
return (
<button className="mt-2 w-full border border-border bg-transparent text-center text-muted-foreground" style={{ borderRadius: 6, padding: '6px 12px', fontSize: 12, opacity: 0.5, cursor: 'not-allowed' }} disabled>+ Link existing</button>
<button className="mt-2 w-full border border-border bg-transparent text-center text-muted-foreground" style={{ borderRadius: 6, padding: '6px 12px', fontSize: 12, opacity: 0.5, cursor: 'not-allowed' }} disabled>+ Add relationship</button>
)
}