fix(ui): close drilldown stack before following FK links

navigateToLink was switching the top-level tab without closing the
detail stack, so the user stayed trapped in the detail view with only
a status message. Now it calls closeAllDetails() first so the link
actually lands on the target tab. Also dropped the noisy "Followed
link to ID N" status message -- cursor placement is sufficient feedback.

Author: cpcloud

internal/app/detail_test.go

@@ -635,3 +635,24 @@ func TestDrilldownHint(t *testing.T) {
 			"kind=%v title=%s", tt.kind, tt.title)
 	}
 }
+
+func TestNavigateToLinkClosesDetailStack(t *testing.T) {
+	m := newTestModelWithDemoData(t, 42)
+	m.active = tabIndex(tabVendors)
+
+	vendors, err := m.store.ListVendors(false)
+	require.NoError(t, err)
+	require.NotEmpty(t, vendors)
+
+	// Drill into vendor quotes.
+	require.NoError(t, m.openVendorQuoteDetail(vendors[0].ID, vendors[0].Name))
+	require.True(t, m.inDetail())
+
+	// Follow the Project link from the detail view.
+	link := &columnLink{TargetTab: tabProjects}
+	require.NoError(t, m.navigateToLink(link, 1))
+
+	// Detail stack should be fully collapsed and we should be on Projects.
+	assert.False(t, m.inDetail(), "detail stack should be closed after navigateToLink")
+	assert.Equal(t, tabIndex(tabProjects), m.active)
+}

internal/app/model.go

@@ -925,18 +925,18 @@ func (m *Model) startCellOrFormEdit() error {
 	return tab.Handler.InlineEdit(m, meta.ID, col)
 }
 
-// navigateToLink switches to the target tab and selects the row matching the FK.
+// navigateToLink closes any open drilldown stack, switches to the target tab,
+// and selects the row matching the FK.
 func (m *Model) navigateToLink(link *columnLink, targetID uint) error {
+	m.closeAllDetails()
 	m.switchToTab(tabIndex(link.TargetTab))
 	tab := m.activeTab()
 	if tab == nil {
 		return fmt.Errorf("target tab not found")
 	}
-	if selectRowByID(tab, targetID) {
-		m.setStatusInfo(fmt.Sprintf("Followed link to ID %d.", targetID))
-		return nil
+	if !selectRowByID(tab, targetID) {
+		m.setStatusError(fmt.Sprintf("Linked item %d not found (deleted?).", targetID))
 	}
-	m.setStatusError(fmt.Sprintf("Linked item %d not found (deleted?).", targetID))
 	return nil
 }