fix(ui): accurate topology filter count + Helm chart-name tooltip#825
Merged
Conversation
- Topology filter sidebar footer: derive both the visible and total counts from the filterable kinds. It previously divided the per-kind visible sum by nodes.length, which includes the synthetic Internet node, so the two never reconciled. Also surface a "· N filtered" indicator with a hover breakdown of which kinds are hidden. - Helm ChartBrowser: add a native title on the truncated chart-name headings so the full name is readable on hover.
This was referenced May 29, 2026
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4954cf7. Configure here.
Reveal truncated chart names with the shared Tooltip component (theme- matched, instant) rather than the browser's native title. Non-conflicting wrapper classes (min-w-0 flex-1 / w-full) keep the truncation working without overriding the Tooltip's default display, so no tailwind-merge dependency or shared-primitive change is needed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small, self-contained UX fixes, reimplemented cleanly off current main.
Topology filter footer count
The "Showing N of M resources" footer summed the visible count over the filterable kinds but used
nodes.lengthas the denominator — andnodes.lengthincludes the syntheticInternetnode, which isn't a filterable kind. So the two never reconciled, and the count could look wrong / a stale "filtered" state could appear with nothing actually hidden. Both numerator and denominator now come from the filterable kinds, and a· N filteredindicator (with a hover breakdown of which kinds are hidden) is shown only when something is actually filtered out.Helm chart-name tooltip
Long Helm chart names are truncated in the catalogue with no way to read the full string. Added a native
titleon the truncated headings so the full name shows on hover — no new dependency, no shared-component change.Notes
These supersede the salvageable parts of #589 (its topology-footer fix) and #577 (its chart-name hover), reworked from scratch rather than carried over — the count helper is inlined (no separate module) and the chart tooltip uses a native
titleinstead of wrapping the sharedTooltipprimitive / adding atailwind-mergedependency. Verified via/visual-test;make tscclean.Note
Low Risk
Display-only UI changes with no auth, data, or API impact.
Overview
Fixes two small UX issues in the topology filter sidebar and Helm chart browser.
The topology “Showing N of M resources” footer now uses the same filterable-kind totals for both numbers (instead of
nodes.length, which included the non-filterable synthetic Internet node). When kind filters hide resources, it shows a · N filtered hint and a hover breakdown of hidden kinds.In ChartBrowser, truncated chart titles in local and ArtifactHub cards are wrapped in the shared
Tooltipso the full chart name is readable on hover.Reviewed by Cursor Bugbot for commit 63564ff. Bugbot is set up for automated code reviews on this repo. Configure here.