fix(gordon): empty-state scroll and wheel-chaining bugs#25475
Conversation
- Make outer messagesContainer the sole always-on scroller and use my-auto on the empty-state root so the robot's top stays at scrollTop=0 and is always reachable, even on short viewports. - Add overscroll-behavior: contain to the drawer paper and messages scroller so the wheel doesn't chain out to the docs page behind. - Drop the robotFloat keyframe (re-rasterises the SVG every frame while idle) and size the robot with clamp(80px, 35%, 160px) so it stays compact on short drawers.
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The three bugs described in the PR — robot cut off on short viewports, wheel-event chaining to the page behind the drawer, and unnecessary SVG rasterization per frame — are addressed by targeted, correct changes:
- Scroll chaining fixed by adding
overscroll-containto the outer chat panel wrapper. - Robot visibility fixed by switching the messages container to a proper
flex min-h-0 flex-1 flex-col overflow-y-autostructure (the standard CSS pattern for scrollable flex children), replacing the previous conditional:classtoggle. - Robot float animation removed along with its
@keyframes robotFloatblock, eliminating the per-frame SVG re-rasterization. - Welcome message layout correctly uses
my-autoon a direct flex child for vertical centering — this works as intended given the constrained flex parent.
No documentation style violations, no Alpine.js x-show+hidden anti-pattern, and no vendored/generated content touched. The fix is surgical and correct.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes to layouts/_partials/gordon-chat.html are clean layout and CSS fixes. No user-facing text was modified, no Hugo template syntax issues were introduced, and no style guide violations are present. The scroll-containment, layout, and animation removal are all appropriate and well-scoped.
3 participants
Description
Three bugs in the Gordon drawer on short viewports (small screens)
scrollTop = 0and can't be scrolled into view — you see the suggestions but not the robot.position: fixed; wheel events over its non-scrolling regions (header, form, disclaimer) or at the messages scroller's boundary chain out to the page underneath.Before
Screen.Recording.2026-07-01.at.15.48.26.mov
After
Screen.Recording.2026-07-01.at.15.49.10.mov
Related issues or tickets
None.
Reviews