-
Notifications
You must be signed in to change notification settings - Fork 481
Resizable left sidebar #459
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
base: main
Are you sure you want to change the base?
Resizable left sidebar #459
Conversation
Make left sidebar resizable so long URLs can be viewed in their entirety. Fixes modelcontextprotocol#457 Vibe-coded with Cursor
@@ -4,6 +4,9 @@ The MCP inspector is a developer tool for testing and debugging MCP servers. | |||
|
|||
 | |||
|
|||
> **Tip:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure that adding this to the Readme should be necessary. If users can't discover the resize feature, that suggests we might need a better cue in the UI like a more obvious drag handle. So lets just leave this out of the doc for now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me and thanks a lot for adding this! Just had one comment on the doc addition.
Hi @msabramo! Making the entire sidebar resizable just to show the URL seems... weird. Why not just do a tooltip when hovering that field? |
Personally I have gotten annoyed with editing hidden arguments as once there are more than a few listed, when using stdio. Seems like this would also help that case. |
Fixes #457
Vibe-coded with Cursor
Motivation and Context
Make left sidebar resizable so long URLs can be viewed in their entirety.
Before
Sidebar narrow so can't see full URL
After
Sidebar can be expanded to see full URL
How Has This Been Tested?
and then resize in browser
Breaking Changes
No
Types of changes
Checklist
Additional context