Polish AI assistant footer menu - #5811
Conversation
Preview deploymentsHost Test Results 1 files 1 suites 2h 1m 45s ⏱️ Results for commit 3f1793f. For more details on these errors, see this check. Realm Server Test Results 1 files ±0 1 suites ±0 15m 40s ⏱️ -58s Results for commit 3f1793f. ± Comparison against earlier commit 77f4159. |
3fd70f6 to
f49fa57
Compare
f49fa57 to
15e5032
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 77f4159a59
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| .menu-content::before, | ||
| .menu-content::after, | ||
| .menu-footer::before { | ||
| .menu-content::after { |
There was a problem hiding this comment.
Keep scroll shadows outside the scrolling element
When a skill or model list exceeds the 18.75rem limit, these pseudo-elements are still descendants of the overflow-y: auto .menu-content, so they move with its contents as the user scrolls; making .pill-menu their containing block changes their offsets but does not make them stationary overlays. Consequently, the top and bottom shadows drift away from the visible edges instead of continuing to indicate additional content. Render the shadows on the non-scrolling wrapper or make them sticky.
Useful? React with 👍 / 👎.
Follow up on #5728