Skip to content

ENG-3252: Remove redundant and flickering tooltip from Action Center view#8094

Merged
nreyes-dev merged 2 commits intomainfrom
nreyes/eng-3252
May 5, 2026
Merged

ENG-3252: Remove redundant and flickering tooltip from Action Center view#8094
nreyes-dev merged 2 commits intomainfrom
nreyes/eng-3252

Conversation

@nreyes-dev
Copy link
Copy Markdown
Contributor

@nreyes-dev nreyes-dev commented May 3, 2026

Ticket ENG-3252

Description Of Changes

The full URN tooltip on Action Center monitor field rows duplicated the path already shown in each row's visible breadcrumb, and was reported as flickering while hovering/scrolling. This PR removes it, following the precedent of #7962 (ENG-3428) which removed a similarly redundant/buggy tooltip from the Activity tab.

How it looked before

image

The Cookie_House_BigQuery.prj-bqextract-419116.cms_publisher.users.last_login visibile in the image's tooltip doesn't add any information to the breadcrumb.

Code Changes

  • Removed the <Tooltip title={urn}> wrapper around the breadcrumb in MonitorFieldListItem.tsx.
  • Removed the now-unused Tooltip import from fidesui in the same file.
  • Added a Fixed changelog fragment.

Steps to Confirm

  1. Run the admin UI against a Plus backend with discovery monitor data.
  2. Navigate to Detection & Discovery → Action center, then open a datastore monitor (e.g., a BigQuery monitor with field-level results).
  3. Hover over the breadcrumb paths in the field rows and confirm no URN tooltip appears.
  4. Confirm the breadcrumb itself still renders the dataset / schema / table path inline as before.

Pre-Merge Checklist

  • Issue requirements met
  • All CI pipelines succeeded
  • CHANGELOG.md updated
    • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
    • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
    • Updates unreleased work already in Changelog, no new entry necessary
  • UX feedback:
    • All UX related changes have been reviewed by a designer
    • No UX review needed
  • Followup issues:
    • Followup issues created
    • No followup issues
  • Database migrations:
    • Ensure that your downrev is up to date with the latest revision on main
    • Ensure that your downgrade() migration is correct and works
      • If a downgrade migration is not possible for this change, please call this out in the PR description!
    • No migrations
  • Documentation:
    • Documentation complete, PR opened in fidesdocs
    • Documentation issue created in fidesdocs
    • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
    • No documentation updates required

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Preview May 3, 2026 11:51pm
fides-privacy-center Ignored Ignored May 3, 2026 11:51pm

Request Review

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Title Lines Statements Branches Functions
admin-ui Coverage: 8%
6.28% (2830/44999) 5.53% (1427/25759) 4.42% (588/13285)
fides-js Coverage: 78%
79.39% (2011/2533) 65.99% (1240/1879) 73.09% (345/472)
privacy-center Coverage: 88%
85.97% (331/385) 81.36% (179/220) 78.87% (56/71)

@nreyes-dev nreyes-dev marked this pull request as ready for review May 3, 2026 23:58
@nreyes-dev nreyes-dev requested a review from a team as a code owner May 3, 2026 23:58
@nreyes-dev nreyes-dev requested review from kruulik and removed request for a team May 3, 2026 23:58
Copy link
Copy Markdown
Contributor

@kruulik kruulik left a comment

Choose a reason for hiding this comment

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

👍

@nreyes-dev nreyes-dev added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit a720701 May 5, 2026
51 checks passed
@nreyes-dev nreyes-dev deleted the nreyes/eng-3252 branch May 5, 2026 17:45
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