Skip to content

fix(web): show full namespace on hover in Home bar chart#694

Merged
memtomem merged 1 commit intomainfrom
fix/home-ns-bar-label-tooltip
May 2, 2026
Merged

fix(web): show full namespace on hover in Home bar chart#694
memtomem merged 1 commit intomainfrom
fix/home-ns-bar-label-tooltip

Conversation

@memtomem
Copy link
Copy Markdown
Owner

@memtomem memtomem commented May 2, 2026

Summary

  • Add a title attribute to each row in the Home tab's namespace bar chart so the full namespace string is reachable on hover.
  • Bump app.js?v=94v=95 so the change reaches users behind the disk cache.

Why

.home-bar-label is CSS-clipped at 120 px with text-overflow: ellipsis. Long auto-namespaces like claude:-Users-<user>-Work-<project> truncate to claude:-Users-... with no way to see the rest — users can't identify which project a row belongs to.

The d3 donut override in settings-namespaces.js:479 is not exercised because d3 is never loaded by the SPA (no <script> tag, no module resolution), so the bar chart at app.js:1265 is what every user actually sees.

Out of scope

  • The dead d3 override in settings-namespaces.js is left alone — separate cleanup PR if anyone cares.
  • This is a frontend-only fix; the underlying long auto-namespace strings (e.g. path-derived claude: namespaces) are unchanged.

Test plan

  • uv run ruff check packages/memtomem/src && uv run ruff format --check packages/memtomem/src clean.
  • Manual: load Home tab, hover a long-namespace row, verify the full string appears in the browser's title tooltip.

🤖 Generated with Claude Code

The Home tab's namespace chart uses ``_renderNsChart`` which CSS-clips
``.home-bar-label`` at 120px with ``text-overflow: ellipsis``. Long
auto-namespaces such as ``claude:-Users-<user>-Work-<project>`` get
truncated to ``claude:-Users-...`` with no way to see the rest — users
can't tell which project a row belongs to.

The d3 donut override in ``settings-namespaces.js`` is not exercised
because d3 is never loaded by the SPA, so this fallback path is what
every user actually sees.

Add a ``title`` to the row so the full namespace is recoverable on
hover. Bumps ``app.js?v=94`` → ``v=95`` per the cache-bust convention.

Co-Authored-By: Claude <[email protected]>
@memtomem memtomem merged commit 29fbc53 into main May 2, 2026
8 of 9 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators May 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants