fix(contacts): detail titles, hide Edit, demote stats (PMC-41) #31

Closed

David wants to merge 0 commits from feat/pmc-41-contacts-ui into main

David commented

2026-05-18 04:30:26 +02:00

Owner

Summary

Closes PMC-41 (Contacts/companies UI). PMC-42/43/44/45 were closed in the audit pass; the remaining valid sub-tasks land here as one combined commit since the edits are tightly intertwined on the same two detail pages.

PMC-49 (titles): Company {props.id} / Contact {props.id} instead of hardcoded "Acme Corp" / "Bob Johnson".
PMC-48 (Edit): dropped from both header action bars per F5; no mutation surface to wire to yet.
PMC-46 (Open Tickets stat): dropped the link-blue styling on the stat span; there is no /tickets?company=:id filter to back the apparent affordance.
PMC-47 (sub-tables): the contacts and tickets sub-tables on CompanyDetailPage already wrap names in Link components; ContactDetailPage's sidebar Company already links back to CompanyDetail. Validated as resolved; no code change.

Test plan

cargo check --features web clean
/companies/foo shows "Company foo" header; New Ticket button present, Edit gone
/contacts/bar shows "Contact bar" header; Edit gone
Open Tickets stat renders as plain text, not blue/clickable

🤖 Generated with Claude Code

## Summary Closes PMC-41 (Contacts/companies UI). PMC-42/43/44/45 were closed in the audit pass; the remaining valid sub-tasks land here as one combined commit since the edits are tightly intertwined on the same two detail pages. - PMC-49 (titles): `Company {props.id}` / `Contact {props.id}` instead of hardcoded "Acme Corp" / "Bob Johnson". - PMC-48 (Edit): dropped from both header action bars per F5; no mutation surface to wire to yet. - PMC-46 (Open Tickets stat): dropped the link-blue styling on the stat span; there is no `/tickets?company=:id` filter to back the apparent affordance. - PMC-47 (sub-tables): the contacts and tickets sub-tables on CompanyDetailPage already wrap names in Link components; ContactDetailPage's sidebar Company already links back to CompanyDetail. Validated as resolved; no code change. ## Test plan - [ ] `cargo check --features web` clean - [ ] `/companies/foo` shows "Company foo" header; New Ticket button present, Edit gone - [ ] `/contacts/bar` shows "Contact bar" header; Edit gone - [ ] Open Tickets stat renders as plain text, not blue/clickable 🤖 Generated with [Claude Code](https://claude.com/claude-code)

David added 1 commit

2026-05-18 04:30:26 +02:00

fix(contacts): detail-page titles, hide Edit, demote Open Tickets stat (PMC-46, PMC-47, PMC-48, PMC-49)

Check / clippy + fmt + tests (pull_request) Failing after 8s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

e42460d33c

CompanyDetailPage and ContactDetailPage hardcoded "Acme Corp" / "Bob Johnson" headers regardless of the route, and the Edit header button had no onclick. Switch the headers to "Company {props.id}" / "Contact {props.id}" so navigation reflects the route, and hide the Edit affordance per the F5 pattern until the contacts mutation surface ships.

The Open Tickets statistic on CompanyDetailPage was styled link-blue but was a bare span, so it looked clickable and wasn't. There's no /tickets?company=:id filter to wire it to yet, so drop the blue styling and render it as a plain stat.

PMC-47: the contacts and recent-tickets sub-tables on CompanyDetailPage already wrap names in Link components pointing to ContactDetail / TicketDetail (lines 388-422 / 447-468). ContactDetailPage's sidebar already links its Company back to CompanyDetail. No remaining dead row clicks; sub-task validated as resolved in main.

#PMC-46 State Done
#PMC-47 State Done
#PMC-48 State Done
#PMC-49 State Done