Skip to content

fix(portal): show full task text for legacy runs - #1480

Open
Gile (Gile987) wants to merge 2 commits into
microsoft:mainfrom
Gile987:fix/766-runs-task-tooltip
Open

Gile (Gile987) wants to merge 2 commits into
microsoft:mainfrom
Gile987:fix/766-runs-task-tooltip

Conversation

@Gile987

Copy link
Copy Markdown
Contributor

Summary

Legacy runs without a task-prompt reference display truncated task text
without a hover preview. Add a Radix tooltip showing the full inline task
on hover or keyboard focus, using the existing 200 ms hover delay.

The tooltip is portaled to avoid table clipping. Existing linked prompt
previews remain unchanged. Truncated numeric filter labels also use Radix
tooltips instead of native title attributes.

Related to #766.

Demos

Before

before.webm

Hovering over a legacy run's truncated task does not reveal the full text.

After

after.webm

Hovering over the same task displays its full text in a styled tooltip.

Testing

  • pnpm exec vitest run apps/portal/src/components/TaskPromptBadge.test.tsx
    — all 19 tests passed.
  • pnpm --filter portal exec tsc --noEmit -p tsconfig.app.json
    — passed.
  • pnpm build — passed.
  • pnpm headers:check — passed.
  • Browser verification: full task text, keyboard focus, Escape dismissal,
    viewport positioning, and no additional task-prompt requests.
  • Full-suite audit (pnpm test --maxWorkers=2): 3,189 tests passed;
    seven Windows-worker tests failed because pwsh was unavailable.
  • pnpm lint: failed because ESLint is unavailable in workspace packages.
    CI currently documents this limitation and permits that step to fail.

Documentation and compatibility

Updated docs/architecture/app-design.md to document legacy task previews
and consistent tooltip timing.

No API, database schema, dependency, or upgrade changes.
CLI capabilities are unchanged; this change affects Portal presentation.

Checklist

  • CLI parity: N/A — Portal presentation only.
  • Updated the TaskPromptBadge Storybook story.
  • Database migration: N/A — no schema changes.
  • Lockfile and NOTICE: N/A — no dependency changes.
  • Video showing the behavior before the suggested change.
  • Video showing the behavior after the suggested change.

@cedricvidal Cedric Vidal (cedricvidal) added the community-contribution Pull requests submitted by external community contributors. label Oct 13, 2026
@Gile987

Copy link
Copy Markdown
Contributor Author

Hi. Pardon my impatience. Did anyone take a look at this?

@manekinekko Wassim Chegham (manekinekko) added area: portal Scope web portal, pages, and user-facing components. language: javascript Work involving JavaScript code, tooling, or dependencies. topic: testing Test coverage, test infrastructure, and validation quality. topic: ui Portal, visual presentation, layout, and interface components. topic: ux Usability, interaction flows, and overall user experience. type: bug Incorrect behavior or a regression that needs fixing. type: documentation Documentation additions, corrections, and improvements. labels Nov 6, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for contributing to Scope, Gile (@Gile987), and for your patience while we got to this review! I appreciate the focused fix, the coverage for hover and keyboard focus, and the matching Storybook and documentation updates. Keeping legacy previews fetch-free while preserving existing linked previews is a good approach.

I found two interaction edge cases worth addressing before merging and left inline comments with suggested fixes and regression coverage. The existing 19 component tests and Portal type check passed; I reproduced both gaps with additional focused tests. Thanks again for helping improve this experience!

@Gile987

Copy link
Copy Markdown
Contributor Author

Thank you for contributing to Scope, Gile (Gile (@Gile987)), and for your patience while we got to this review! I appreciate the focused fix, the coverage for hover and keyboard focus, and the matching Storybook and documentation updates. Keeping legacy previews fetch-free while preserving existing linked previews is a good approach.

I found two interaction edge cases worth addressing before merging and left inline comments with suggested fixes and regression coverage. The existing 19 component tests and Portal type check passed; I reproduced both gaps with additional focused tests. Thanks again for helping improve this experience!

Thank you very much.
Hopefully everything is OK now.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: portal Scope web portal, pages, and user-facing components. community-contribution Pull requests submitted by external community contributors. language: javascript Work involving JavaScript code, tooling, or dependencies. topic: testing Test coverage, test infrastructure, and validation quality. topic: ui Portal, visual presentation, layout, and interface components. topic: ux Usability, interaction flows, and overall user experience. type: bug Incorrect behavior or a regression that needs fixing. type: documentation Documentation additions, corrections, and improvements.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants