Skip to content

Fix lobby room list scrolling#259

Merged
coder13 merged 2 commits into
devfrom
agent/fix-lobby-scroll
Jul 18, 2026
Merged

Fix lobby room list scrolling#259
coder13 merged 2 commits into
devfrom
agent/fix-lobby-scroll

Conversation

@coder13

@coder13 coder13 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

What changed

  • constrain the lobby flex hierarchy to the available viewport height
  • keep both room and user panels bounded within the Material UI Grid row
  • add a Cypress regression that forces overflow in a short viewport and verifies real scroll movement

Root cause

The room list declared overflow-y: auto, but flex and Grid ancestors remained content-sized. The room panel therefore grew beyond the viewport instead of giving the list a bounded scroll area; global page overflow then clipped the remaining rooms.

User impact

Users can scroll through all public and private rooms again, including in short and mobile-sized viewports.

Validation

  • yarn workspace letscube-client lint (passes; existing warnings only)
  • yarn workspace letscube-client test:ci --runInBand (97/97)
  • yarn workspace letscube-client build
  • yarn cypress:run --spec cypress/e2e/local_stack.cy.js (11/11)
  • git diff --check
  • full committed diff reviewed with no findings

coder13 added 2 commits July 15, 2026 16:00
Release user-scoped solve history preview
Constrain the lobby flex hierarchy to the viewport so the existing room list overflow can scroll. Add a full-stack browser regression for short viewports.
@coder13
coder13 marked this pull request as ready for review July 18, 2026 00:33
@coder13
coder13 merged commit 92052e3 into dev Jul 18, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant