Skip to content

Eval NUX: replace play overlay with View sample trace button#1900

Merged
chelojimenez merged 2 commits intomainfrom
feat/eval-nux-view-sample-trace-button
Apr 23, 2026
Merged

Eval NUX: replace play overlay with View sample trace button#1900
chelojimenez merged 2 commits intomainfrom
feat/eval-nux-view-sample-trace-button

Conversation

@vigneshgopalandd-maker
Copy link
Copy Markdown
Collaborator

Center the CTA on the first-run sample trace preview; remove the duplicate heading so the subtext alone describes the action.

Made-with: Cursor

Center the CTA on the first-run sample trace preview; remove the duplicate
heading so the subtext alone describes the action.

Made-with: Cursor
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Apr 23, 2026
@chelojimenez
Copy link
Copy Markdown
Contributor

chelojimenez commented Apr 23, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@dosubot dosubot Bot added the enhancement New feature or request label Apr 23, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Internal preview

Preview URL will appear in Railway after the deploy finishes.
Deployed commit: fd207d6
PR head commit: e1af4cc
Backend target: staging fallback.
Access is employee-only in non-production environments.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 23, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 23, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 37981ed4-4ffe-4bbf-8caa-4e1a1cd6f47b

📥 Commits

Reviewing files that changed from the base of the PR and between 7a652c9 and e1af4cc.

📒 Files selected for processing (1)
  • mcpjam-inspector/client/src/components/CiEvalsTab.tsx

Walkthrough

The sample-trace overlay UI in the "no visible suites" empty state of the CiEvalsTab component is restyled. The central call-to-action transitions from an icon-based Play button to a text-only rounded bordered pill labeled "View sample trace". The bottom-left overlay content is consolidated—the separate "View sample trace" heading is removed and replaced with a single condensed description paragraph featuring adjusted opacity, typography, and a max-width constraint. The Play icon import is removed accordingly.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@chelojimenez chelojimenez merged commit 456ef0f into main Apr 23, 2026
10 of 11 checks passed
@chelojimenez chelojimenez deleted the feat/eval-nux-view-sample-trace-button branch April 23, 2026 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants