Skip to content

fix(core): show full job name & id on queue list via tooltip#42

Open
omar-zeineddine wants to merge 1 commit into
pontusab:mainfrom
omar-zeineddine:fix/queue-full-job-names
Open

fix(core): show full job name & id on queue list via tooltip#42
omar-zeineddine wants to merge 1 commit into
pontusab:mainfrom
omar-zeineddine:fix/queue-full-job-names

Conversation

@omar-zeineddine

Copy link
Copy Markdown

Problem

On the queue/overview job list, job ids were hard-truncated to 24 characters (truncate(job.id, 24)) with no way to recover the rest. Custom ids like order_8353ba2431a19308cb… were unreadable — you couldn't tell two jobs apart or copy the full id without opening each one.

Fix

  • Drop the JS truncate(job.id, 24) and render the full id — CSS truncate already ellipsizes it to fit the column responsively.
  • Wrap the name/id block in a Tooltip that reveals the complete job name and full id on hover (break-all so long order_… / pf_… ids wrap legibly).

No new dependencies — uses the existing Tooltip primitive (already provided at the app root via TooltipProvider).

Before / After

Before: order_8353ba2431a19308cb… (cut at 24 chars, full value unreachable)
After: full id shown to column width, complete name + id on hover.

Testing

  • tsc --noEmit
  • biome check
  • vite build (UI bundle) ✅

@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

@omar-zeineddine is attempting to deploy a commit to the Pontus Abrahamsson's projects Team on Vercel.

A member of the Team first needs to authorize it.

@omar-zeineddine omar-zeineddine force-pushed the fix/queue-full-job-names branch from ec5c296 to 6b258b5 Compare June 24, 2026 17:38
The queue list hard-truncated job ids to 24 chars with no way to see
the full value — custom ids like "order_8353…" were unreadable. Drop
the JS truncation (CSS already ellipsizes to fit the column) and wrap
the name/id block in a tooltip that reveals the complete job name and
full id on hover.
@omar-zeineddine omar-zeineddine force-pushed the fix/queue-full-job-names branch from 6b258b5 to 0d9dd56 Compare June 24, 2026 17:40
omarzeineddine-ai added a commit to amalinvest/workbench that referenced this pull request Jul 11, 2026
…m PR pontusab#42)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
omarzeineddine-ai added a commit to amalinvest/workbench that referenced this pull request Jul 11, 2026
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