Skip to content

Fix T3 Sidebar long child-menu scrolling - #2109

Open
Willhong wants to merge 2 commits into
get-bb:mainfrom
Willhong:fix/t3sidebar-child-menu-scroll
Open

Fix T3 Sidebar long child-menu scrolling#2109
Willhong wants to merge 2 commits into
get-bb:mainfrom
Willhong:fix/t3sidebar-child-menu-scroll

Conversation

@Willhong

Copy link
Copy Markdown
Contributor

What was wrong

T3 Sidebar rendered every child in an uncapped menu and an ordinary ul. With enough children, the menu extended beyond the thread viewport, but the list had no scroll range, so lower children were unreachable with both a desktop wheel and compact touch input. Fixes #2105. Standalone counterpart: SawyerHood/bb-plugin-t3sidebar#4.

What changed

  • Cap the child menu at the smaller of 32rem and the available dynamic viewport height.
  • Make the menu a flex column and the child list its min-h-0, overflow-y-auto scroll region.
  • Contain scroll chaining and preserve mobile WebKit touch/momentum scrolling.
  • Add a 26-child regression test that pins the bounded-menu and scroll-container contract.
  • No host-daemon wire, public plugin API, CLI, configuration, guide, or documentation changes.

How you verified

  • pnpm exec turbo run test typecheck --filter=bb-plugin-t3sidebar — 81 tests passed in the scroll-only iteration; typecheck passed.
  • Desktop Chrome at 1728 × 813, 26 children:
    • before: menu 1,157px; list maxScroll 0; wheel 500px left scrollTop = 0
    • after: menu 512px; list maxScroll 645px; wheel 500px moved scrollTop = 500
  • The regression test fails on the old classes and passes with this patch.

Fixes #2105

AGENT GENERATED: by GPT-5.6

@Willhong

Copy link
Copy Markdown
Contributor Author

Standalone implementation counterpart: SawyerHood/bb-plugin-t3sidebar#6

@Willhong
Willhong marked this pull request as ready for review August 20, 2026 23:43
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.

T3 Sidebar child-thread menu cannot scroll long lists

1 participant