Skip to content

Comments

Fix(webapp): logs button + logs table row link fix#3107

Merged
samejr merged 7 commits intomainfrom
fix(webapp)-logs-button
Feb 20, 2026
Merged

Fix(webapp): logs button + logs table row link fix#3107
samejr merged 7 commits intomainfrom
fix(webapp)-logs-button

Conversation

@samejr
Copy link
Member

@samejr samejr commented Feb 20, 2026

Small fixes and improvements to the logs page:

  • Clicking the Run ID didn't open inspector
  • Swapped the "open link in tab" icon with Runs icon
  • Prevent tooltip hovering on Level info
CleanShot 2026-02-20 at 10 00 37@2x

@changeset-bot
Copy link

changeset-bot bot commented Feb 20, 2026

⚠️ No Changeset found

Latest commit: 67c6e1a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 20, 2026

Walkthrough

The pull request introduces UI refinements and a new tooltip control feature across three components. A new optional prop disableTooltipHoverableContent is added to the Table component's header cell to control tooltip behavior, which is then utilized in LogsTable. The Run link button in LogsTable is visually updated with a different trailing icon (RunsIcon replacing ArrowTopRightOnSquareIcon) and adjusted styling for text sizing. Additionally, the Run ID cell is made clickable, and a button variant in LogDetailView is changed from tertiary/small to secondary/small for different visual appearance.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the key changes with bullet points and includes a helpful screenshot, but it lacks required template sections (Closes #, Testing, Changelog) and the checklist is not completed. Add the missing issue reference, complete the checklist, provide testing steps, and add a formal changelog entry following the repository template.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main changes: fixing the logs button styling and logs table row link behavior, which aligns with the actual modifications across three files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix(webapp)-logs-button

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.

@samejr samejr changed the title Fix(webapp) logs button Fix(webapp): logs button + logs table row link fix Feb 20, 2026
@samejr samejr marked this pull request as ready for review February 20, 2026 16:02
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@samejr samejr merged commit 79f8cde into main Feb 20, 2026
30 checks passed
@samejr samejr deleted the fix(webapp)-logs-button branch February 20, 2026 16:44
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