Skip to content

Show script ID/configuration profile UUID on hover in UI#45787

Open
noahtalerman wants to merge 1 commit into
mainfrom
tooltips-ids
Open

Show script ID/configuration profile UUID on hover in UI#45787
noahtalerman wants to merge 1 commit into
mainfrom
tooltips-ids

Conversation

@noahtalerman
Copy link
Copy Markdown
Member

@noahtalerman noahtalerman commented May 19, 2026

For the following quick win:

Screenshots

Screenshot 2026-05-19 at 8 59 36 AM Screenshot 2026-05-19 at 8 59 25 AM

Summary by CodeRabbit

  • New Features
    • Hovering over profile names now displays the profile UUID in a tooltip
    • Hovering over script names now displays the script ID in a tooltip

Review Change Stack

@noahtalerman noahtalerman requested a review from a team as a code owner May 19, 2026 13:04
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@noahtalerman
Copy link
Copy Markdown
Member Author

Screenshots

Screenshot 2026-05-19 at 8 59 36 AM Screenshot 2026-05-19 at 8 59 25 AM

@mike-j-thomas new UI pattern. What do you think?

@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.74%. Comparing base (1e22a58) to head (b7678c8).
⚠️ Report is 58 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #45787      +/-   ##
==========================================
- Coverage   66.74%   66.74%   -0.01%     
==========================================
  Files        2744     2744              
  Lines      219316   219384      +68     
  Branches    10794    10837      +43     
==========================================
+ Hits       146393   146436      +43     
- Misses      59698    59722      +24     
- Partials    13225    13226       +1     
Flag Coverage Δ
frontend 55.61% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 19, 2026

Walkthrough

This PR adds hover tooltips to display identifiers in two list-item components. ProfileListItem now shows the profile UUID when hovering over the profile name, and ScriptListItem displays the script ID when hovering over the script name. Both changes follow the same pattern: importing TooltipWrapper, wrapping the existing title element, and configuring the tooltip with the identifier content and top-position styling.

Possibly related issues

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description links to the related issue and includes screenshots demonstrating the feature, but is missing most required checklist items from the template. Complete the PR description template by checking appropriate boxes and confirming testing, database, and compatibility considerations were addressed.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title directly and clearly describes the main change: adding tooltips that display script IDs and configuration profile UUIDs on hover, which matches the implementation across both modified files.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tooltips-ids

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.

@noahtalerman noahtalerman changed the title Show script ID and configuration profile UUID on hover over name Show script ID/configuration profile UUID on hover in UI May 19, 2026
@mike-j-thomas
Copy link
Copy Markdown
Member

@noahtalerman, the tooltip itself looks good, but how would a user know to spawn it?

Is the top example that's underlined a link? That would stop us from being able to use the dotted underline. So what would it look like with a question mark in a circle icon next to the label?

@noahtalerman
Copy link
Copy Markdown
Member Author

Is the top example that's underlined a link?

@mike-j-thomas yes.

So what would it look like with a question mark in a circle icon next to the label?

It's going to become a long string of misaligned icons which I think will look strange. Picture it for this list in dogfood:

Screenshot 2026-05-19 at 9 27 35 AM

how would a user know to spawn it?

I hear what you're saying but maybe it's ok?

GitHub has no indicator until I hover:

Screenshot 2026-05-19 at 9 26 11 AM Screenshot 2026-05-19 at 9 26 08 AM

Mike, up to you.

@mike-j-thomas
Copy link
Copy Markdown
Member

It's going to become a long string of misaligned icons

More than the misalignment, it's going to become very dense for something that may not be that important (don't actually have context for how important it is to show the UUID).

I say go for it, at least for now. Besides, everyone likes a good easter egg.

@georgekarrv
Copy link
Copy Markdown
Member

I would say this is functionally not helpful. Most of the time when you want an id or uuid you want to copy and paste it. Tooltips don't allow you to accomplish that

@noahtalerman
Copy link
Copy Markdown
Member Author

Most of the time when you want an id or uuid you want to copy and paste it. Tooltips don't allow you to accomplish that

@georgekarrv copy/paste works!

Screenshot 2026-05-19 at 10 37 16 AM

@MagnusHJensen
Copy link
Copy Markdown
Member

Maybe not part of this, but doing something similar (wherever this lands) for the host UUID would be nice. I always find myself copying it from the network request when trying to hit it via the API.

@noahtalerman
Copy link
Copy Markdown
Member Author

Maybe not part of this, but doing something similar (wherever this lands) for the host UUID would be nice. I always find myself copying it from the network request when trying to hit it via the API.

@MagnusHJensen good idea. Where in the UI makes sense to add the tooltip? On the Hosts page?

I guess on the Hosts page, I can already copy/paste the UUID from the table...

Screenshot 2026-05-20 at 9 18 06 AM

@MagnusHJensen
Copy link
Copy Markdown
Member

Maybe not part of this, but doing something similar (wherever this lands) for the host UUID would be nice. I always find myself copying it from the network request when trying to hit it via the API.

@MagnusHJensen good idea. Where in the UI makes sense to add the tooltip? On the Hosts page?

I guess on the Hosts page, I can already copy/paste the UUID from the table...

Screenshot 2026-05-20 at 9 18 06 AM

Well of course, then never mind for that, I just didn’t use the UUID column 😅

@mike-j-thomas mike-j-thomas removed their request for review May 22, 2026 07:37
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.

4 participants