Skip to content

fix(issues): Derive activity timeline connectors from layout - #121578

Merged
scttcper merged 2 commits into
masterfrom
scttcper/activity-rollup-connector
Aug 7, 2026
Merged

fix(issues): Derive activity timeline connectors from layout#121578
scttcper merged 2 commits into
masterfrom
scttcper/activity-rollup-connector

Conversation

@scttcper

@scttcper scttcper commented Aug 7, 2026

Copy link
Copy Markdown
Member

fixes missing activity timeline connectors around collapsed activity rollups

image

Collapsed activity rows could drop the trailing timeline connector because each component had to manually forward whether another row followed it.

Let CSS derive the connector from sibling position and remove the showConnector prop plumbing entirely.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Story previews

Preview the stories changed in this PR on the Vercel deployment:

Preview deployment: https://sentry-9tw9cqywp.sentry.dev

Update the progress activity tooltip to use the shared sibling-based connector styling too.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@scttcper
scttcper requested a review from a team August 7, 2026 23:28
@scttcper
scttcper marked this pull request as ready for review August 7, 2026 23:29
@scttcper
scttcper requested a review from a team as a code owner August 7, 2026 23:29
@scttcper
scttcper enabled auto-merge (squash) August 7, 2026 23:29
@scttcper
scttcper disabled auto-merge August 7, 2026 23:29
@scttcper
scttcper merged commit b815e2e into master Aug 7, 2026
77 checks passed
@scttcper
scttcper deleted the scttcper/activity-rollup-connector branch August 7, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants