fix(webapp): restore admin debug tooltip on Tasks and Runs, make its IDs copyable#4332
Conversation
…IDs copyable The admin-only debug tooltip (the shield icon in the page header) was removed from the Tasks page in #3941 and had never been present on the Runs list. Re-add `<AdminDebugTooltip />` to both pages' accessories so it matches the other dashboard pages. Also wrap the tooltip's identifier values (User ID, Org ID, Project ID, Project ref, Environment ID) in `CopyableText` so admins can copy them on hover, reusing the same copy-on-hover control used in table cells.
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughThe debug panel now uses a controlled popover with delayed close handling and renders identifiers through 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
The copy button is absolutely positioned to the right of each value (`-right-6`), which pushed it past the popover's right edge where `overflow-y-auto` clipped it. Add right padding so it always shows.
…ns work Radix tooltips enforce one-open-at-a-time globally (opening any tooltip dispatches a document event that closes the others), so hovering a copy button — which opens its own "Copy" tooltip — dismissed the debug panel. Render the panel as a hover-triggered Popover instead. A popover holds interactive content without closing, so copy-on-hover now works and the panel stays open while you interact with it.
The custom rows some pages add to the admin debug panel (Run/Deployment IDs, trace id, env/org ids, image reference, region/branch ids, etc.) were still plain text. Wrap the opaque identifiers in CopyableText across those pages so they match the shared rows and can be copied on hover. Enum/boolean/human-readable rows stay plain (Environment type/paused, deployment Platform, plan name) and the Build Server link is left as-is. Also drop a duplicated project ID that rendered twice in the env settings panel.
…ad of a Popover Add a hideTooltip prop to CopyableText that renders the copy control without its own tooltip. The admin debug panel passes it on every copy control, which removes the nested tooltip that was firing Radix's global 'one tooltip open at a time' close. With the root cause gone the panel reverts from a Popover back to a plain Tooltip (no manual open/close timer or effect needed).
Restores the debug panel on the Tasks and Runs pages, and makes the data it shows copyable.
Admin/impersonation only — no change for regular users, so there's no
.server-changes