Skip to content

fix(inspector): keep tab count badges visible in collapsed mode#1490

Merged
khandrew1 merged 3 commits into
mcp-use:canaryfrom
Arpitm544:fix/inspector-tab-badge-visibility-1480-clean
May 19, 2026
Merged

fix(inspector): keep tab count badges visible in collapsed mode#1490
khandrew1 merged 3 commits into
mcp-use:canaryfrom
Arpitm544:fix/inspector-tab-badge-visibility-1480-clean

Conversation

@Arpitm544
Copy link
Copy Markdown
Contributor

@Arpitm544 Arpitm544 commented May 13, 2026

Closes #1480. This PR refactors TabsTrigger to use a generic 'badge' prop and ensures it remains visible even when the tab bar is collapsed/icon-only. (Cleaned up from previous PR #1489)

Screenshot 2026-05-13 at 12 25 42 PM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 13, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@mcp-use/cli@1490
npm i https://pkg.pr.new/create-mcp-use-app@1490
npm i https://pkg.pr.new/@mcp-use/inspector@1490
npm i https://pkg.pr.new/mcp-use@1490

commit: 955bcb6

@khandrew1 khandrew1 assigned Arpitm544 and unassigned Arpitm544 May 13, 2026
Copy link
Copy Markdown
Contributor

@khandrew1 khandrew1 left a comment

Choose a reason for hiding this comment

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

Image

I just reviewed this PR, and not sure why the badge shifts sides when selected. it should just always be on the right.

On another note this might cause the width to be too wide and cause that weird overflow on the top bar. Do you have any thoughts on that?

@Arpitm544
Copy link
Copy Markdown
Contributor Author

Arpitm544 commented May 14, 2026

Thanks for giving the feedback I shift the badge from

icon -> badge -> label
to
icon -> label -> badge

so it stays on the right consistently
Screenshot 2026-05-15 at 12 26 43 AM

Yeah good point . Badges make the tabs a little wider, but I see that tab list already support horizontal scrolling if it gets too wide. If need , we can make the badges smaller or shorter in larger counts later like 99+

Copy link
Copy Markdown
Contributor

@khandrew1 khandrew1 left a comment

Choose a reason for hiding this comment

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

Also add a changeset. Thanks! I think this is better UX for now.

Comment thread libraries/typescript/packages/inspector/src/client/components/LayoutHeader.tsx Outdated
Comment thread libraries/typescript/packages/inspector/src/client/components/LayoutHeader.tsx Outdated
@Arpitm544 Arpitm544 force-pushed the fix/inspector-tab-badge-visibility-1480-clean branch from 145719c to 955bcb6 Compare May 19, 2026 22:12
@khandrew1 khandrew1 changed the base branch from main to canary May 19, 2026 22:24
@khandrew1 khandrew1 merged commit 64e2ae3 into mcp-use:canary May 19, 2026
65 of 69 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inspector: Keep numeric tab badge visible in collapsed (icon-only) header

2 participants