Skip to content

Extract shared component detail view#2317

Draft
Mbeaulne wants to merge 1 commit into
05-27-natural-language-rerank-servicefrom
05-27-shared-component-detail
Draft

Extract shared component detail view#2317
Mbeaulne wants to merge 1 commit into
05-27-natural-language-rerank-servicefrom
05-27-shared-component-detail

Conversation

@Mbeaulne
Copy link
Copy Markdown
Collaborator

@Mbeaulne Mbeaulne commented May 27, 2026

Tophatting

Manual tophatting recommended because this changes component detail rendering.

  1. Open the existing Components page.
  2. Select a component.
  3. Confirm the detail view still shows metadata, inputs, outputs, and source code.

What changed

Extracts the component detail view into a shared component.

This lets both the existing Components page and the new Components V2 page show the same component details without duplicating the rendering code.

Why

The new search page needs a detail panel. Sharing the detail view keeps the UI consistent and avoids maintaining two versions of the same layout.

Test plan

  • Open the existing Components page
  • Select a component and confirm details still render
  • Confirm inputs, outputs, metadata, and source code still appear
  • Run pnpm run typecheck --pretty false

Copy link
Copy Markdown
Collaborator Author

Mbeaulne commented May 27, 2026

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 27, 2026

🎩 Preview

A preview build has been created at: 05-27-shared-component-detail/452da91

@Mbeaulne Mbeaulne force-pushed the 05-27-shared-component-detail branch from 860f34b to c5b0341 Compare May 27, 2026 20:07
@Mbeaulne Mbeaulne force-pushed the 05-27-agent-settings-ai-rerank-service branch 2 times, most recently from aa37795 to 6d8a2b9 Compare May 27, 2026 20:16
@Mbeaulne Mbeaulne force-pushed the 05-27-shared-component-detail branch from c5b0341 to 61f4e01 Compare May 27, 2026 20:16
@Mbeaulne Mbeaulne changed the base branch from 05-27-agent-settings-ai-rerank-service to graphite-base/2317 May 27, 2026 20:27
@Mbeaulne Mbeaulne force-pushed the 05-27-shared-component-detail branch from 61f4e01 to 62c24f9 Compare May 27, 2026 20:27
@Mbeaulne Mbeaulne force-pushed the graphite-base/2317 branch from 6d8a2b9 to 582c2ef Compare May 27, 2026 20:27
@Mbeaulne Mbeaulne changed the base branch from graphite-base/2317 to 05-27-natural-language-rerank-service May 27, 2026 20:28
@Mbeaulne Mbeaulne force-pushed the 05-27-natural-language-rerank-service branch from 582c2ef to 6b1d4bc Compare May 27, 2026 20:29
@Mbeaulne Mbeaulne force-pushed the 05-27-shared-component-detail branch from 62c24f9 to 29c5e4e Compare May 27, 2026 20:29
@Mbeaulne Mbeaulne force-pushed the 05-27-shared-component-detail branch from 29c5e4e to 452da91 Compare May 27, 2026 23:18
@Mbeaulne Mbeaulne force-pushed the 05-27-natural-language-rerank-service branch from 6b1d4bc to 0213639 Compare May 27, 2026 23:18
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.

1 participant