BuzzbeeSCD/openchamber
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(settings): add always-visible scrollbar to settings sub-panels

Browse Source 
Replace overflow-hidden with overflow-y-scroll on 5 content wrapper divs
so Settings sub-panels (Shortcuts, Plugins, etc.) have an always-visible
vertical scrollbar on both desktop and mobile.

Fixes #2475
This commit is contained in:
Sérgio Pedro
2026-07-28 18:28:25 +00:00
parent d50cb5becc
commit 9acbebe189
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/ui/src/components/views/SettingsView.tsx
+5 -5
View File
@@ -1013,13 +1013,13 @@ export const SettingsView: React.FC<SettingsViewProps> = ({ onClose, forceMobile
// No sidebar available; fall back to direct content.
const fallback = renderPageContent(settingsSlug);
return (
<div className="flex-1 min-h-0 overflow-hidden bg-background">
<div className="flex-1 min-h-0 overflow-y-scroll bg-background">
<ErrorBoundary>{fallback}</ErrorBoundary>
</div>
);
}
return (
<div className="flex-1 min-h-0 overflow-hidden bg-background">
<div className="flex-1 min-h-0 overflow-y-scroll bg-background">
<ErrorBoundary>
{renderPageSidebar(settingsSlug, { onItemSelect: handleMobilePageSidebarItemSelect })}
</ErrorBoundary>
@@ -1031,7 +1031,7 @@ export const SettingsView: React.FC<SettingsViewProps> = ({ onClose, forceMobile
const content = renderPageContent(settingsSlug);
return (
<div className="flex-1 min-h-0 overflow-hidden bg-background">
<div className="flex-1 min-h-0 overflow-y-scroll bg-background">
<ErrorBoundary>{content}</ErrorBoundary>
</div>
);
@@ -1048,7 +1048,7 @@ export const SettingsView: React.FC<SettingsViewProps> = ({ onClose, forceMobile
<div className={cn('border-r', runtimeCtx.isVSCode ? 'bg-background' : 'bg-sidebar')} style={{ width: SETTINGS_SPLIT_SIDEBAR_WIDTH, minWidth: SETTINGS_SPLIT_SIDEBAR_WIDTH, borderColor: 'var(--interactive-border)' }}>
<ErrorBoundary>{renderPageSidebar(settingsSlug, {})}</ErrorBoundary>
</div>
<div className="flex-1 min-h-0 overflow-hidden bg-background">
<div className="flex-1 min-h-0 overflow-y-scroll bg-background">
<ErrorBoundary>{renderPageContent(settingsSlug)}</ErrorBoundary>
</div>
</div>
@@ -1056,7 +1056,7 @@ export const SettingsView: React.FC<SettingsViewProps> = ({ onClose, forceMobile
}
return (
<div className="h-full min-h-0 overflow-hidden bg-background">
<div className="h-full min-h-0 overflow-y-scroll bg-background">
<ErrorBoundary>{renderPageContent(settingsSlug)}</ErrorBoundary>
</div>
);