fix(settings): let fixed-width controls scale with font size and density (#2739)

fix(settings): let fixed-width controls scale with font size and density
This commit is contained in:
Bohdan Triapitsyn
2026-08-28 23:45:27 +03:00
committed by GitHub
6 changed files with 19 additions and 13 deletions
@@ -450,7 +450,7 @@ export const AgentsPage: React.FC = () => {
inputMode="decimal" inputMode="decimal"
placeholder="—" placeholder="—"
emptyLabel="—" emptyLabel="—"
className="w-16" className="w-20"
/> />
{temperature !== undefined && ( {temperature !== undefined && (
<Button <Button
@@ -488,7 +488,7 @@ export const AgentsPage: React.FC = () => {
inputMode="decimal" inputMode="decimal"
placeholder="—" placeholder="—"
emptyLabel="—" emptyLabel="—"
className="w-16" className="w-20"
/> />
{topP !== undefined && ( {topP !== undefined && (
<Button <Button
@@ -3,6 +3,7 @@ import { Button } from '@/components/ui/button';
import { Textarea } from '@/components/ui/textarea'; import { Textarea } from '@/components/ui/textarea';
import { toast } from '@/components/ui'; import { toast } from '@/components/ui';
import { useI18n, type I18nKey } from '@/lib/i18n'; import { useI18n, type I18nKey } from '@/lib/i18n';
import { cn } from '@/lib/utils';
import { reportSettingsSaveState } from '@/lib/persistence'; import { reportSettingsSaveState } from '@/lib/persistence';
import { useIsVSCodeRuntime } from '@/hooks/useRuntimeAPIs'; import { useIsVSCodeRuntime } from '@/hooks/useRuntimeAPIs';
import { import {
@@ -375,7 +376,7 @@ export const BehaviorPage: React.FC = () => {
onValueChange={(value) => setResponseStylePreset(value)} onValueChange={(value) => setResponseStylePreset(value)}
disabled={isLoading || !responseStyleEnabled} disabled={isLoading || !responseStyleEnabled}
> >
<SelectTrigger size={SETTINGS_SELECT_SIZE} className={SETTINGS_SELECT_ROW_TRIGGER_CLASS}> <SelectTrigger size={SETTINGS_SELECT_SIZE} className={cn(SETTINGS_SELECT_ROW_TRIGGER_CLASS, 'max-w-72')}>
<SelectValue> <SelectValue>
{(value) => { {(value) => {
if (value === 'custom') return t('settings.behavior.page.responseStyle.option.custom'); if (value === 'custom') return t('settings.behavior.page.responseStyle.option.custom');
@@ -1198,7 +1198,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
controlClassName="w-full" controlClassName="w-full"
> >
<Select value={uiFont} onValueChange={(value) => setUiFont(value as UiFontOption)}> <Select value={uiFont} onValueChange={(value) => setUiFont(value as UiFontOption)}>
<SelectTrigger aria-label={t('settings.openchamber.visual.field.selectInterfaceFontAria')} size={SETTINGS_SELECT_SIZE} className={SETTINGS_SELECT_TRIGGER_CLASS}> <SelectTrigger aria-label={t('settings.openchamber.visual.field.selectInterfaceFontAria')} size={SETTINGS_SELECT_SIZE} className={cn(SETTINGS_SELECT_TRIGGER_CLASS, 'max-w-full')}>
<SelectValue>{UI_FONT_OPTIONS.find((option) => option.id === uiFont)?.label}</SelectValue> <SelectValue>{UI_FONT_OPTIONS.find((option) => option.id === uiFont)?.label}</SelectValue>
</SelectTrigger> </SelectTrigger>
<SelectContent> <SelectContent>
@@ -1228,7 +1228,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
controlClassName="w-full" controlClassName="w-full"
> >
<Select value={monoFont} onValueChange={(value) => setMonoFont(value as MonoFontOption)}> <Select value={monoFont} onValueChange={(value) => setMonoFont(value as MonoFontOption)}>
<SelectTrigger aria-label={t('settings.openchamber.visual.field.selectCodeFontAria')} size={SETTINGS_SELECT_SIZE} className={SETTINGS_SELECT_TRIGGER_CLASS}> <SelectTrigger aria-label={t('settings.openchamber.visual.field.selectCodeFontAria')} size={SETTINGS_SELECT_SIZE} className={cn(SETTINGS_SELECT_TRIGGER_CLASS, 'max-w-full')}>
<SelectValue>{CODE_FONT_OPTIONS.find((option) => option.id === monoFont)?.label}</SelectValue> <SelectValue>{CODE_FONT_OPTIONS.find((option) => option.id === monoFont)?.label}</SelectValue>
</SelectTrigger> </SelectTrigger>
<SelectContent> <SelectContent>
@@ -1269,6 +1269,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
min={50} min={50}
max={200} max={200}
step={5} step={5}
className="w-20"
aria-label={t('settings.openchamber.visual.field.fontSizePercentageAria')} aria-label={t('settings.openchamber.visual.field.fontSizePercentageAria')}
/> />
<span className={SETTINGS_NUMBER_UNIT_CLASS}>%</span> <span className={SETTINGS_NUMBER_UNIT_CLASS}>%</span>
@@ -1299,6 +1300,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
min={9} min={9}
max={52} max={52}
step={1} step={1}
className="w-20"
/> />
<span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span> <span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span>
<Button size="sm" <Button size="sm"
@@ -1328,6 +1330,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
min={9} min={9}
max={32} max={32}
step={1} step={1}
className="w-20"
/> />
<span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span> <span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span>
<Button size="sm" <Button size="sm"
@@ -1362,6 +1365,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
min={50} min={50}
max={200} max={200}
step={5} step={5}
className="w-20"
/> />
<span className={SETTINGS_NUMBER_UNIT_CLASS}>%</span> <span className={SETTINGS_NUMBER_UNIT_CLASS}>%</span>
<Button size="sm" <Button size="sm"
@@ -1392,6 +1396,7 @@ export const OpenChamberVisualSettings: React.FC<OpenChamberVisualSettingsProps>
min={0} min={0}
max={100} max={100}
step={5} step={5}
className="w-20"
/> />
<span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span> <span className={SETTINGS_NUMBER_UNIT_CLASS}>px</span>
<Button size="sm" <Button size="sm"
@@ -87,7 +87,7 @@ export const SessionRetentionSettings: React.FC = () => {
max={MAX_DAYS} max={MAX_DAYS}
step={1} step={1}
aria-label={t('settings.openchamber.sessionRetention.field.retentionPeriodAria')} aria-label={t('settings.openchamber.sessionRetention.field.retentionPeriodAria')}
className="w-20 tabular-nums" className="w-24 tabular-nums"
/> />
<span className="typography-ui-label text-muted-foreground">{t('settings.openchamber.sessionRetention.field.days')}</span> <span className="typography-ui-label text-muted-foreground">{t('settings.openchamber.sessionRetention.field.days')}</span>
<Button <Button
@@ -1051,13 +1051,13 @@ export const VoiceSettings: React.FC = () => {
{/* Speech Rate */} {/* Speech Rate */}
<SettingsFieldRow label={t('settings.voice.page.field.speechRate')}> <SettingsFieldRow label={t('settings.voice.page.field.speechRate')}>
{!isMobile && <input type="range" min={0.5} max={2} step={0.1} value={speechRate} onChange={(e) => setSpeechRate(Number(e.target.value))} className={sliderClass} />} {!isMobile && <input type="range" min={0.5} max={2} step={0.1} value={speechRate} onChange={(e) => setSpeechRate(Number(e.target.value))} className={sliderClass} />}
<NumberInput value={speechRate} onValueChange={setSpeechRate} min={0.5} max={2} step={0.1} className="w-16 tabular-nums" /> <NumberInput value={speechRate} onValueChange={setSpeechRate} min={0.5} max={2} step={0.1} className="w-20 tabular-nums" />
</SettingsFieldRow> </SettingsFieldRow>
{/* Speech Pitch */} {/* Speech Pitch */}
<SettingsFieldRow label={t('settings.voice.page.field.speechPitch')}> <SettingsFieldRow label={t('settings.voice.page.field.speechPitch')}>
{!isMobile && <input type="range" min={0.5} max={2} step={0.1} value={speechPitch} onChange={(e) => setSpeechPitch(Number(e.target.value))} className={sliderClass} />} {!isMobile && <input type="range" min={0.5} max={2} step={0.1} value={speechPitch} onChange={(e) => setSpeechPitch(Number(e.target.value))} className={sliderClass} />}
<NumberInput value={speechPitch} onValueChange={setSpeechPitch} min={0.5} max={2} step={0.1} className="w-16 tabular-nums" /> <NumberInput value={speechPitch} onValueChange={setSpeechPitch} min={0.5} max={2} step={0.1} className="w-20 tabular-nums" />
</SettingsFieldRow> </SettingsFieldRow>
{/* Speech Volume */} {/* Speech Volume */}
@@ -2322,7 +2322,7 @@ export const RemoteInstancesPage: React.FC = () => {
min={5} min={5}
max={240} max={240}
step={1} step={1}
className="w-16 tabular-nums" className="w-20 tabular-nums"
value={draft.connectionTimeoutSec} value={draft.connectionTimeoutSec}
onValueChange={(next) => { onValueChange={(next) => {
updateDraft((current) => ({ updateDraft((current) => ({
@@ -2350,7 +2350,7 @@ export const RemoteInstancesPage: React.FC = () => {
min={1} min={1}
max={65535} max={65535}
step={1} step={1}
className="w-20 tabular-nums" className="w-32 tabular-nums"
value={draft.remoteOpenchamber.preferredPort} value={draft.remoteOpenchamber.preferredPort}
onValueChange={(next) => { onValueChange={(next) => {
updateDraft((current) => ({ updateDraft((current) => ({
@@ -2517,7 +2517,7 @@ export const RemoteInstancesPage: React.FC = () => {
min={1} min={1}
max={65535} max={65535}
step={1} step={1}
className="w-20 tabular-nums" className="w-32 tabular-nums"
value={draft.localForward.preferredLocalPort} value={draft.localForward.preferredLocalPort}
onValueChange={(next) => { onValueChange={(next) => {
updateDraft((current) => ({ updateDraft((current) => ({
@@ -2775,7 +2775,7 @@ export const RemoteInstancesPage: React.FC = () => {
min={1} min={1}
max={65535} max={65535}
step={1} step={1}
className="w-16 tabular-nums" className="w-32 tabular-nums"
value={forward.localPort} value={forward.localPort}
onValueChange={(next) => { onValueChange={(next) => {
updateForward((item) => ({ updateForward((item) => ({
@@ -2817,7 +2817,7 @@ export const RemoteInstancesPage: React.FC = () => {
min={1} min={1}
max={65535} max={65535}
step={1} step={1}
className="w-16 tabular-nums" className="w-32 tabular-nums"
value={forward.remotePort} value={forward.remotePort}
onValueChange={(next) => { onValueChange={(next) => {
updateForward((item) => ({ updateForward((item) => ({