Conversation
Signed-off-by: Peter Jausovec <[email protected]>
Signed-off-by: Peter Jausovec <[email protected]>
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refreshes the UI to a cleaner layout with dark/light theming, improved navigation, and a sheet-based details experience for registry entities (servers/skills/agents/prompts) and deployed resources.
Changes:
- Adds theme support (dark/light) and updates global styling + typography.
- Reworks Catalog cards and details views, moving details into a Radix-based sheet.
- Updates Deployed Resources page with search + filtering and a redesigned list layout.
Reviewed changes
Copilot reviewed 21 out of 25 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| ui/tailwind.config.ts | Adds a new sans font family configuration to align Tailwind typography with the new app font. |
| ui/public/logo-light.svg | Adds a new light-mode SVG logo asset. |
| ui/public/logo-dark.svg | Adds a new dark-mode SVG logo asset. |
| ui/public/arlogo.png | Adds/updates a raster logo asset used by the UI. |
| ui/package.json | Adds Radix Dialog and baseline-browser-mapping dependency updates to support the new sheet and Next tooling. |
| ui/package-lock.json | Locks dependency updates (Next + transitive updates) to match the refreshed UI dependencies. |
| ui/components/ui/sheet.tsx | Introduces a Sheet component (Radix Dialog-based) used for entity detail panels. |
| ui/components/theme-provider.tsx | Adds a NextThemes ThemeProvider wrapper for class-based theming. |
| ui/components/skill-detail.tsx | Redesigns skill detail presentation for sheet usage and adds “copy raw JSON”. |
| ui/components/skill-card.tsx | Redesigns skill list row/card styling and actions. |
| ui/components/server-detail.tsx | Redesigns server detail presentation for sheet usage and adds “copy raw JSON”. |
| ui/components/server-card.tsx | Redesigns server list row/card styling and actions. |
| ui/components/prompt-detail.tsx | Redesigns prompt detail presentation for sheet usage and adds “copy raw JSON”. |
| ui/components/prompt-card.tsx | Redesigns prompt list row/card styling. |
| ui/components/navigation.tsx | Updates navbar layout and adds a theme toggle + theme-aware logo. |
| ui/components/footer.tsx | Adds a new footer with theme-aware logo and external links. |
| ui/components/agent-detail.tsx | Redesigns agent detail presentation for sheet usage. |
| ui/components/agent-card.tsx | Redesigns agent list row/card styling. |
| ui/components/tests/skill-card.test.tsx | Updates tests to reflect new SkillCard rendering. |
| ui/components/tests/server-card.test.tsx | Updates tests to reflect new ServerCard rendering. |
| ui/app/page.tsx | Rebuilds the Catalog UI with a tab bar, search/filters, empty states, and sheet-based details. |
| ui/app/layout.tsx | Wires in ThemeProvider + Footer and switches font to Plus Jakarta Sans. |
| ui/app/globals.css | Updates design tokens and base styles for the refreshed theme. |
| ui/app/deployed/page.tsx | Redesigns deployed resources UI with search, filters, and a compact row layout. |
| ui/.storybook/preview-head.html | Loads Plus Jakarta Sans in Storybook and sets the font CSS variable. |
Files not reviewed (1)
- ui/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Peter Jausovec <[email protected]>
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
Signed-off-by: Peter Jausovec <[email protected]>
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
inFocus7
reviewed
Mar 12, 2026
Collaborator
inFocus7
left a comment
inFocus7
left a comment
There was a problem hiding this comment.
Going to review code now (at least to the best of my ability, but leaving some general UX feedback after playing around with the UI a bit):
- [UI] On the import server modal, the blue background used for the "Common Registry URLs" helper doesn't feel like it follows the same design scheme as the rest of the site
- [Backend?] When I tried to import https://registry.modelcontextprotocol.io/v0.1/servers, I get a 404 on the http://localhost:12121/v0/import endpoint
- There's an import-dialog.tsx file where we're fetching /v0/import , I'm not entirely sure if we actually export the import server functionality as an HTTP endpoint :thinking-duck:
- We could probably create a feature for its support through an endpoint (assuming I'm right on my findings that the endpoint DNE)
- [UI Nitpick] on the modals (e.g. Add New MCP Server) the dropdown (e.g. Repository URL) should have some padding on the right side of the down chevron. It's too pushed against the border.
- [UI Nitpick] on the modals (e.g. Add New MCP Server) the radio buttons used (e.g. in "Transport") have text that is very small.
- [UI] We should add padding to some of the modals (e.g. Add New MCP Server) namely on the horizontal side. Without an "X" exit button, since there is no padding on the horizontal sides, it's a bit annoying clicking out.
- [UX inconsistency] In the Prompt catalog, there's not way to see different versions of the same prompt (either in the catalog list, or detailed view), whereas for other resources (at least agents), you can see multiple versions of the same agent. This is inconsistent, although I'm not sure if it's on purpose (maybe the idea is "who cares about older versions of prompts")
- [UX Nitpick] The catalog Add button is a bit hard to see / the black text on dark purple background has little contrast. Not sure what the right colors here should be though.
inFocus7
reviewed
Mar 12, 2026
Collaborator
There was a problem hiding this comment.
Aside from my initial feedback #324 (review) everything else looks good. Happy to approve if you decide against them (or after doing updates) 👍🏼
Signed-off-by: Peter Jausovec <[email protected]>
Contributor
Author
|
fixed the UI issues -- logged #335 for the import functionality. Make the version display the same across all artifacts - we grouped them by version, in the details, user can switch versions. 
|
|
You already have 3 pull requests open. Please consider working on getting the existing ones merged before opening new ones. Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
cleans up the ui, add dark/light mode, change the way we display item details (using a sheet, instead of dialog), cleans up the display of deployed items, adds filters, search, etc.
Screen.Recording.2026-03-12.at.11.09.39.AM.mov
Change Type
Changelog