Skip to content

fix: align built-in mcp detail page header with standard mcp detail page#2013

Merged
adaam2 merged 3 commits intomainfrom
fix/built-in-mcp-details-header
Mar 27, 2026
Merged

fix: align built-in mcp detail page header with standard mcp detail page#2013
adaam2 merged 3 commits intomainfrom
fix/built-in-mcp-details-header

Conversation

@adaam2
Copy link
Copy Markdown
Member

@adaam2 adaam2 commented Mar 27, 2026

Summary

  • Replace custom image-based hero header on the built-in MCP detail page with the shared DetailHero component (dotted pattern background) to match the standard MCP detail page
  • Update install page section styling (bordered container, primary View button, CodeBlock inner classes) to match the normal MCP detail page
  • Remove redundant Configuration section that duplicates content already available on the install page
  • Delete unused MCPHeroIllustration component from SourceCardIllustrations.tsx

Test plan

  • Navigate to a built-in MCP detail page and verify the header now shows the dotted pattern background matching standard MCP detail pages
  • Verify the "Built-in" badge and URL copy button render correctly in the hero
  • Verify the Install Page section has the bordered container with primary View button
  • Navigate to a standard MCP detail page and confirm it still looks correct
  • Confirm no TypeScript or lint errors

🤖 Generated with Claude Code


Open with Devin

Use shared DetailHero component instead of custom image-based hero,
match install page section styling, and remove redundant configuration
section that duplicates the install page.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adaam2 adaam2 requested a review from a team as a code owner March 27, 2026 10:58
@adaam2 adaam2 added the bug Something isn't working label Mar 27, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 27, 2026

🦋 Changeset detected

Latest commit: fb17c0c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dashboard Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gram-docs-redirect Ready Ready Preview, Comment Mar 27, 2026 11:06am

Request Review

@adaam2 adaam2 changed the title fix: align built-in MCP detail page header with standard MCP detail page fix: align built-in mcp detail page header with standard mcp detail page Mar 27, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 2 additional findings in Devin Review.

Open in Devin Review

@adaam2 adaam2 merged commit 2b7754e into main Mar 27, 2026
31 checks passed
@adaam2 adaam2 deleted the fix/built-in-mcp-details-header branch March 27, 2026 11:09
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants