Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make record name editable on show page #9172

Merged
merged 2 commits into from
Dec 26, 2024

Conversation

thomtrp
Copy link
Contributor

@thomtrp thomtrp commented Dec 20, 2024

When fields are not displayed in show page, title should be editable

Enregistrement.de.l.ecran.2024-12-20.a.16.04.24.mov

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR adds record name editing functionality in the show page header and breadcrumb navigation, with conditional rendering based on field visibility.

  • Added new RecordEditableName component in /pages/object-record/RecordEditableName.tsx for inline name editing with proper loading states
  • Modified RecordShowPageHeader to conditionally render editable name when layout.hideSummaryAndFields is true
  • Made Icon prop optional in NavigationDrawerInput and NavigationDrawerItem components to support breadcrumb editing
  • Added Storybook story with GraphQL mocks to test RecordEditableName rendering and interactions
  • Exposed objectMetadataItem through useRecordShowPagePagination hook to support metadata access for name editing

8 file(s) reviewed, 6 comment(s)
Edit PR Review Bot Settings | Greptile

@thomtrp thomtrp force-pushed the tt-make-record-name-editable-in-breadcumb branch 2 times, most recently from 7c19f5e to 9a5a87e Compare December 20, 2024 17:28
@thomtrp thomtrp force-pushed the tt-make-record-name-editable-in-breadcumb branch from b2c2e2e to d470edf Compare December 26, 2024 09:13
Copy link
Contributor

@bosiraphael bosiraphael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bosiraphael bosiraphael merged commit f63b0a2 into main Dec 26, 2024
22 checks passed
@bosiraphael bosiraphael deleted the tt-make-record-name-editable-in-breadcumb branch December 26, 2024 09:59
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.

2 participants