[Bug] Settings page left plugin nav overflows with no scrollbar — items clipped & unclickable

[Hanhaijl]

Environment

• DSH version: 0.1.1-rc.2
• Reproduction: any deployment where enough plugins register a settings.section entry.
• This is a core DSH UI bug, not a plugin issue.

Problem

The Settings page is a centered modal panel. Its left rail lists General / Models / Plugins / Agent Presets plus one entry per plugin that registers a settings.section. When the number of entries grows, the nav list overflows the panel bottom:

• entries beyond the fold are clipped and unclickable
• there is no scrollbar at all

So a user who installs many plugins literally cannot reach the bottom entries in the Settings nav.

Root cause

File: packages/client/ui-settings-general/src/client/SettingsRoot.module.css (compiled class names shown).

• .panel (→ .VOzbGW_panel): display:flex; ...; overflow:hidden
• .nav (→ .VOzbGW_nav): flex-direction:column; ...; display:flex — no overflow-y
• .navList (→ .VOzbGW_navList): flex-direction:column;gap:4px;display:flex — no flex growth / no scroll

Because the panel is overflow:hidden and the nav list neither grows nor scrolls, when .navList content exceeds the panel height the extra entries are simply cut off and no scrollbar appears.

Suggested fix

Let the nav list grow and scroll while keeping the panel title fixed:

.navList {
  flex-direction: column;
  gap: 4px;
  flex: 1;
  min-height: 0;
  overflow-y: auto;
  display: flex;
}

• flex: 1 lets the list fill the remaining height of the stretched nav column
• min-height: 0 allows it to shrink below its content (the flexbox scroll requirement)
• overflow-y: auto shows a scrollbar only when entries overflow

With few entries the layout is unchanged (no regression — list is top-aligned with empty space below, exactly as before). The nav rail is 188px wide, which already works for this pattern. Optionally add a small padding-bottom to .navList so the last item is not flush to the bottom edge when scrolled fully down.

I verified this on 0.1.1-rc.2: adding the above to .navList makes the Settings nav scroll correctly and all entries become reachable.

[denial123789]

Confirmed against rc.2 source at b150a55: the report identifies the correct scroll-owner gap. The content column already uses the complete pattern on .options; the nav list does not.

The important part is keeping all three declarations together on .navList: flex: 1 allocates the remaining column height, min-height: 0 allows the flex child below min-content size, and overflow-y: auto creates the bounded scrollport. Scrolling .nav would move the Settings title; unclipping or scrolling .panel would couple both columns and the close action.

I would add a many-section regression that proves scrollHeight > clientHeight, scrolls the final button into view, activates it by pointer and keyboard, and checks short viewport / 200% zoom. The test should target an owned locator rather than a generated CSS-module class.

Detailed source trace, containment, test matrix, and partial-fix router: https://github.com/sandbaseai/deepseek-harness-handbook/blob/main/docs/en/troubleshooting/settings-plugin-nav-overflow.md

[Jstn-1g]

I implemented and independently verified the minimal scroll-owner fix against official master b150a551b8d465e31e418e1b2eaf5e79bbb7d28e:

• exact commit: Jstn-1g@4c87c90
• reviewable branch: https://github.com/Jstn-1g/deepseek-harness/tree/reference/discussion-4729-settings-nav-scroll

The patch keeps the Settings title outside the scrollport and adds the complete flex: 1; min-height: 0; overflow-y: auto contract only to .navList.

The assembled-browser regression uses the four shipping section rows at a 900×260 viewport. It traverses backward from Close to the final row, proves focus scrolls that row fully inside the nav list, then activates it with Enter. Rebuilding the prior CSS is a causal negative control: scrollHeight === clientHeight and the list is clipped by the panel; the exact commit produces a bounded overflow scrollport and passes.

Verification at the exact head: focused browser regression 1/1; settings package 44/44; scrollbar contracts 21/21; independent full GUI run 3,997 passed / 1 skipped; client typecheck, package/web builds, full lint, bilingual docs, translation pairing, links, change-scope, and diff checks all passed.

[Zn-Dk]

Quick status check — I'm hitting the exact same issue on current official master.
I just synced to dd6322d6, and SettingsRoot.module.css still defines .navList with only display: flex; flex-direction: column; gap: 4px — no flex: 1, no min-height: 0, no overflow-y: auto — while .options keeps the full scroll-owner pattern. So the diagnosis in the first comment still holds on master.

Is there an open PR for the fix yet, or is it still in progress? Happy to test/review a branch if one is available.