Show script ID on hover/tooltip in the UI#45772
Conversation
Show the script's API ID as a native browser tooltip (title attribute) when hovering over script names. This surfaces the ID users need for API calls without requiring them to use the API or browser network inspector. Applies to: - Controls > Scripts library list - Host > Run Script modal table - Batch Run Script list Closes #45771
There was a problem hiding this comment.
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.
mikermcneil
left a comment
There was a problem hiding this comment.
This checks out to me at a glance, but I'm not familiar enough with React syntax to give this a useful "Approve". Will leave to y'all. Thanks!
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #45772 +/- ##
=======================================
Coverage 66.76% 66.76%
=======================================
Files 2747 2747
Lines 219724 219724
Branches 10845 10845
=======================================
Hits 146693 146693
Misses 59775 59775
Partials 13256 13256
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
titleattribute) showing the script's API ID when users hover over script names in the UIAPI id: <script_id>Changes
frontend/pages/ManageControlsPage/Scripts/components/ScriptListItem/ScriptListItem.tsx— Addedtitleprop to the Button wrapping the script namefrontend/pages/hosts/details/HostDetailsPage/modals/RunScriptModal/ScriptsTableConfig.tsx— Addedtitleprop to the Button in the script name columnfrontend/pages/hosts/ManageHostsPage/components/RunScriptBatchPaginatedList/RunScriptBatchPaginatedList.tsx— Addedtitleattribute to the anchor elementTesting
Hover over any script name in the UI to see the tooltip displaying
API id: <number>.Closes #45771 and #24320
Built for Mike McNeil by Kilo for Slack