Skip to content

fix: full width in Chrome - #581

Merged
cliffhall merged 3 commits into
modelcontextprotocol:mainfrom
richardkmichael:fix-chrome-width
Jul 2, 2025
Merged

fix: full width in Chrome#581
cliffhall merged 3 commits into
modelcontextprotocol:mainfrom
richardkmichael:fix-chrome-width

Conversation

@richardkmichael

Copy link
Copy Markdown
Contributor

Motivation and Context

Chrome does not display the Inspector UI full-width, but is instead centered.

How Has This Been Tested?

Verified in latest Chrome and Firefox.

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

Reported by @cliffhall in a PR comment

@richardkmichael

richardkmichael commented Jul 1, 2025

Copy link
Copy Markdown
Contributor Author

@cliffhall I think I got this right, but you might want to check it. Resize works naturally now too.

Screenshot 2025-07-01 at 09 41 23

@cliffhall

Copy link
Copy Markdown
Member

Tested locally and it works for me, but then I can't dup on my mac with FF, Safari, or Chrome. I've asked @olaservo and @evalstate if they can give it a go, since they're both able to see the issue.

@cliffhall cliffhall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 👍

@cliffhall
cliffhall merged commit ff1e5ec into modelcontextprotocol:main Jul 2, 2025
4 checks passed
@richardkmichael
richardkmichael deleted the fix-chrome-width branch August 1, 2025 20:37
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.

2 participants