Skip to content

fix(web): chat text no longer shows through a 1px gap above the composer context strip - #5

Open
jagrat7 wants to merge 1 commit into
mainfrom
fix/composer-context-strip-seam
Open

jagrat7 wants to merge 1 commit into
mainfrom
fix/composer-context-strip-seam

Conversation

@jagrat7

@jagrat7 jagrat7 commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

When a notice (for example "Server update available") is attached above the composer, a single row of unblurred chat text shows through at the seam between the collapsed input and the context strip below it.

With an attached notice, the composer shell drops its shared glass backdrop, so the input and the context strip each paint their own. The strip's glass was masked to start exactly at the input's bottom edge. When that edge falls off the device-pixel grid, Chromium drops the last row of the input's filtered backdrop, which leaves a one-pixel gap. This is the same failure pingdotgg#10635 fixed for the banner seam.

The fix moves the strip's mask cut-off 1px higher, so its glass starts under the input. The input paints above the strip with its own glass, so the overlap is never visible.

🤖 Generated with Claude Code — Claude Opus 5.5 (1M context)


Devin Review

RetriggerConfidence Score: 5/5

The PR appears safe to merge.

Summary

The PR moves the context strip’s glass-mask cutoff one pixel under the composer to cover a seam that can expose unblurred chat text when a notice is attached.

  • No new actionable issue was identified in the PR change.

Reviews (2) · Last reviewed commit: "fix(web): chat text no longer shows thro..."

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Sep 30, 2026

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Thread transfer impact

⚠️ The latest CI run did not produce a thread transfer result for 92f39a0.

This comment will update automatically after the next completed run.

…ser context strip

With a notice attached above the composer, the input and the context strip
each paint their own glass. The strip's mask started exactly at the input's
bottom edge, and Chromium drops the last device-pixel row of the input's
filtered backdrop when that edge is off the pixel grid, leaving one row of
unblurred chat text visible. Bleed the strip's mask 1px under the input,
matching the fix already applied to attached banners.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jagrat7
jagrat7 force-pushed the fix/composer-context-strip-seam branch from c9bd089 to 92f39a0 Compare October 3, 2026 07:04

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

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant