Repository navigation
Conversation
rdimitrov
force-pushed
the
fix/ui-prefer-server-title
branch
from
November 12, 2026 15:20
21ddf2b to
39749d9
Compare
rdimitrov
approved these changes
Nov 12, 2026
rdimitrov
left a comment
Member
There was a problem hiding this comment.
Reviewed the rebased change with icons preserved in both normal and Recently Updated cards. The optional title improves readability while the full namespaced name remains visible; missing or identical titles fall back without duplication. Search semantics are unchanged.
Browser checks passed at 320, 390, 768, 1024, and 1440 px, including long titles and names, absent/broken icons, HTML escaping, expanded details, and hiding recent cards during search. Fresh CI passed: https://github.com/modelcontextprotocol/registry/actions/runs/43246733127.
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.
Summary
server.titleas the primary heading in Registry cardsserver.namevisible as a smaller secondary identity when the values differserver.nameheading whentitleis absent or identicalWhy
The Registry schema defines
titleas a human-readable display name, but collapsed cards currently headline only the reverse-DNSname. That makes browsing harder for users who do not recognize programmatic MCP identifiers.Generic example:
The technical identity remains visible for namespace verification. All user-provided values continue to pass through the existing
escapeHtml(...)helper, and long values use the existing Tailwind wrapping utilities.Scope
This is presentation-only and backward-compatible. It does not change schemas, API responses, search behavior, publishing, authentication, or protocol behavior.
The API
searchparameter currently matches onlyserver_name; title and description search are not included here. Icon rendering is also intentionally out of scope and is already tracked by #784.Testing
git diff --checkLocal
make checkwas unavailable because this Windows environment does not have Go, Make, Docker, or a WSL distribution installed. The upstream PR CI is configured to run lint, schema validation, build, vulnerability scanning, and all tests.