Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

Fix Home "Running" strip cards clipped on hover - #3683

Closed
puemos wants to merge 1 commit into
mainfrom
posthog-code/fix-running-card-hover-clip
Closed

Fix Home "Running" strip cards clipped on hover#3683
puemos wants to merge 1 commit into
mainfrom
posthog-code/fix-running-card-hover-clip

Conversation

@puemos

@puemos puemos commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Problem

Why: A task card in the Home "Running" agents strip was visibly clipped — the top of the card (and its title text) got cut off on hover.

The cards lift up on hover (hover:-translate-y-px + hover:shadow-sm) as an affordance, but they sat flush against the top of the horizontal ScrollArea, which clips vertical overflow. So the moment a card lifted, its top edge and title were cropped.

Changes

Add top padding (pt-1) to the strip's scroll content in HomeActiveAgentsStrip.tsx so the lifted card and its shadow stay inside the ScrollArea's clip box. One-line className change; the hover animation is preserved.

How did you test this?

  • pnpm --filter @posthog/ui typecheck — passes.
  • biome check on the changed file — clean.

Visual QA in the running app (hover a running card, confirm the top is no longer clipped) still recommended before merge.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

The cards in the Home "Running" agents strip lift up on hover (hover:-translate-y-px + hover:shadow-sm), but they sat flush against the top of the horizontal ScrollArea, which clips vertical overflow. On hover the card's top edge and title text were cropped.

Add top padding (pt-1) to the scroll content so the lifted card and its shadow stay inside the ScrollArea's clip box, preserving the hover affordance.

Generated-By: PostHog Code
Task-Id: 97ebead6-4f61-4859-908e-a2048e74180e
@trunk-io

trunk-io Bot commented Jul 22, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit f4a2a00.

@puemos puemos closed this Jul 22, 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.

1 participant