fix: commands popup hidden under UI elements (#936)

[NoahHahm]

Summary

• Removes paint from contain property on .chat-composer-shell in src/index.css
• contain: paint was creating a new stacking context that clipped the commands popup, causing it to render behind other UI elements

Root Cause

contain: paint tells the browser the element's content won't overflow its bounds, which implicitly creates a new stacking context. This prevented the commands popup from appearing above sibling elements regardless of z-index.

Fix

/* before */
contain: layout style paint;

/* after */
contain: layout style;

Test plan

• Type / in chat input → commands popup appears above all UI elements
• Click the commands button → popup renders correctly
• Verify on mobile and web interface
• Confirm regression from v1.34.0 → v1.35.0 is resolved

Closes #936

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Improved chat composer rendering behavior by adjusting how the interface is isolated by the browser, which may help reduce visual glitches or repaint issues.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: fcf1e793-4642-4a4e-9b02-b183e7770cef

📥 Commits

Reviewing files that changed from the base of the PR and between 2ebe64f and 6fb3ec0.

📒 Files selected for processing (1)

• src/index.css

---

📝 Walkthrough

Walkthrough

In src/index.css, the contain property on .chat-composer-shell is changed from layout style paint to layout style, removing the paint containment that was causing the commands popup to be clipped by the browser's paint isolation boundary.

Changes

Commands Popup Visibility Fix

Layer / File(s)	Summary
Remove paint containment src/index.css	contain on .chat-composer-shell drops the paint keyword, allowing the commands popup to render outside the element's paint boundary.

A bunny hopped through the CSS one day,
Found paint containment blocking the way,
The popup was hidden, clipped out of sight,
One keyword removed — now it renders right!
🐇✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the popup layering fix and matches the main change.
Linked Issues check	✅ Passed	The CSS change directly addresses #936 by removing paint containment that hid the commands popup.
Out of Scope Changes check	✅ Passed	The PR contains only the targeted CSS containment change, with no unrelated additions.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}