fix(resources): align Read Resource button position with Run button#1917
fix(resources): align Read Resource button position with Run button#1917nuthalapativarun wants to merge 1 commit intoMCPJam:mainfrom
Conversation
Move 'ml-auto' from the secondary actions wrapper to the Read Resource button itself, matching the layout pattern used in ToolsSidebar where the execute button sits at the far right of the header row. Before: [Tabs] ........ [Refresh][Hide] [Read Resource] After: [Tabs] [Refresh][Hide] ............. [Read Resource] Fixes MCPJam#1415
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe ResourcesTab component adjusts Tailwind spacing utilities for button alignment within its sidebar. The wrapper element loses its Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes #1415
The Read Resource button for resource templates was placed after the secondary action buttons (Refresh, Hide sidebar), while the Run button in the Tools tab uses
ml-autoto sit at the far right of the header — a pattern users expect for the primary execute action.Before:
[Tabs] ........ [Refresh][Hide] [Read Resource]After:
[Tabs] [Refresh][Hide] ............. [Read Resource]Changes:
ml-autofrom the secondary actions wrapper div inResourcesTab.tsxml-autoto the Read Resource button so it anchors to the far right, matchingToolsSidebar.tsxTesting
npx -y @modelcontextprotocol/server-everything)Unit tests: all 15 existing
ResourcesTabtests pass.