fix(web): remove glass chrome strip below chat composer#3706
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want fixes drafted automatically? Bugbot Autofix can create code changes for findings. A team admin can enable Autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4a342fb. Configure here.
| ref={setComposerOverlayElement} | ||
| data-chat-composer-overlay="true" | ||
| className="pointer-events-none absolute inset-x-0 bottom-0 z-20 pt-1.5 sm:pt-2" | ||
| className="pointer-events-none absolute inset-x-0 bottom-0 z-20 pt-1.5 pb-[calc(env(safe-area-inset-bottom)+0.75rem)] sm:pt-2 sm:pb-[calc(env(safe-area-inset-bottom)+1rem)]" |
There was a problem hiding this comment.
Bottom safe-area taps hit timeline
Medium Severity
Bottom spacing moved onto the pointer-events-none composer overlay, but the removed chat-composer-lower-chrome sibling was the only in-flow layer in that band with default pointer hit-testing. Taps in the safe-area and extra bottom padding can reach the scrolling timeline underneath instead of being absorbed above it.
Reviewed by Cursor Bugbot for commit 4a342fb. Configure here.
ApprovabilityVerdict: Needs human review This UI cleanup removes a glass effect below the chat composer. However, an unresolved review comment flags a potential pointer-events issue where taps in the bottom safe-area may now pass through to the timeline underneath, which warrants human verification. You can customize Macroscope's approvability policy. Learn more. |


What Changed
Removed the glass chrome strip that rendered below the chat composer: the empty
chat-composer-lower-chromediv and thechat-composer-shared-blurbackdrop layer, plus their now-dead CSS rules. The composer overlay keeps the same bottom spacing via its own padding, and the composer'sfloating-glass-surfaceis unchanged.Why
The strip appeared as a second glass box leaking out under the input, instead of the composer floating on its own. Deleting the two decorative elements and folding their shared background values into the surviving
.floating-glass-surfaceselectors fixes it with a net −29 line diff and no behavior change to the input itself.UI Changes
Checklist
Note
Low Risk
Visual/layout-only chat composer chrome cleanup with no logic or API changes; minor risk of bottom spacing differences on devices with safe-area insets.
Overview
Fixes a second glass panel appearing under the chat input by removing the decorative backdrop layer (
chat-composer-shared-blur) and the emptychat-composer-lower-chromespacer fromChatView.Bottom safe-area padding now lives on the composer overlay wrapper instead of the removed lower chrome, so timeline inset /
composerOverlayHeightbehavior should stay aligned.floating-glass-surfaceonChatComposeris unchanged; related CSS is trimmed so glass styles apply only to that class (no shared rules with the deleted chrome).Reviewed by Cursor Bugbot for commit 4a342fb. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Remove glass chrome strip below chat composer
Removes the lower chrome divider and shared blur wrapper elements from the chat composer in ChatView.tsx. Safe-area bottom padding is now applied directly on the composer overlay wrapper. Corresponding CSS classes
chat-composer-lower-chromeandchat-composer-shared-blurare deleted from index.css, with glass styling consolidated onto.floating-glass-surfaceonly.Macroscope summarized 4a342fb.