Skip to content

improvement(chat): reuse shared overflow fades on scroll edges - #8390

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/chat-scroll-fade
Sep 28, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
codex/chat-scroll-fade

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Add conditional top and bottom fades to the shared workspace, organization, and Copilot chat transcript.
  • Reuse the OverflowText fade width for chat and standalone search results, keeping the composer controls outside the mask.

Type of Change

  • Improvement

Testing

  • Cleanup review: no findings.
  • App type-check and 14 existing tests across five nearby chat suites passed.
  • Lint, block registry audit, full audit suite, and docs manifest verification passed.
  • Regenerated committed artifacts; no generated changes.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 28, 2026 9:21pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Adds visual scroll-edge fades to two chat and search UI components.

The PR appears safe to merge; no actionable issue was identified.

Summary

The PR applies shared conditional scroll-edge fades to chat transcripts and aligns standalone search results with the shared fade width.

  • Chat observes the virtualized transcript and fades only edges with hidden content.
  • Search results use the shared 16px fade band.

Reviews (1) · Last reviewed commit: "improvement(chat): reuse shared overflow..."

@waleedlatif1
waleedlatif1 merged commit 59ee92b into staging Sep 28, 2026
23 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/chat-scroll-fade branch September 28, 2026 21:29

This branch was previously deployed

1 inactive deployment
Preview — bed58cd8 Deployed Sep 28, 2026 by vercel[bot]
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.

1 participant