Skip to content

Restore Options dialog tree/section chrome to capture scale #320

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
baseGame merged 1 commit into from
Sep 22, 2026
Merged

Restore Options dialog tree/section chrome to capture scale #320

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
95 changes: 95 additions & 0 deletions src/styles/main.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3660,3 +3660,98 @@ html[data-show-path-bars='0'] .path-pair-bar {
padding: 0 5px !important;
font-size: 11px !important;
}

/* Options / Settings dialog chrome at capture CSS scale (tree rows ~18px; controls ~20px) */
.settings-view {
gap: 6px !important;
padding: 4px 6px !important;
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .options-section-nav {
gap: 4px !important;
padding: 2px 4px !important;
border: 1px solid #a0a0a0 !important;
border-radius: 0 !important;
}

.settings-view .options-tree-group-label {
padding: 4px 6px 2px !important;
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .options-section-button {
min-height: 18px !important;
padding: 2px 6px !important;
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .options-hint {
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .stack-row input,
.settings-view .stack-row select,
.settings-view .open-with-draft input {
height: 20px !important;
min-height: 20px !important;
padding: 0 6px !important;
border: 1px solid #a0a0a0 !important;
border-radius: 0 !important;
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .auto-save-limit-row,
.settings-view .font-size-row,
.settings-view .tweak-row {
gap: 6px !important;
min-height: 20px !important;
line-height: 16px !important;
}

.settings-view .shortcut-row,
.settings-view .shared-session-list li {
min-height: 20px !important;
padding: 2px 6px !important;
border: 1px solid #a0a0a0 !important;
border-radius: 0 !important;
}

.settings-view .n-card {
--n-padding-top: 4px !important;
--n-padding-bottom: 4px !important;
--n-padding-left: 6px !important;
--n-padding-right: 6px !important;
--n-title-font-size: 12px !important;
--n-border-radius: 0 !important;

border: 1px solid #a0a0a0 !important;
border-radius: 0 !important;
font-size: 11px !important;
line-height: 16px !important;
}

.settings-view .n-card-header {
min-height: 20px !important;
padding: 2px 6px !important;
font-size: 12px !important;
line-height: 16px !important;
}

.settings-view .n-input,
.settings-view .n-base-selection,
.settings-view .n-input-number,
.settings-view .n-button {
--n-height: 20px !important;
--n-font-size: 11px !important;
--n-border-radius: 0 !important;

min-height: 20px !important;
border-radius: 0 !important;
line-height: 16px !important;
}
27 changes: 21 additions & 6 deletions src/styles/peekOptionsDensity.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,13 +61,28 @@ describe('peek panel densify', () => {
})

describe('options dialog densify', () => {
it('uses a dense tree/content Options layout and keeps wired section ids', () => {
expect(settingsView).toMatch(/\.settings-view\s*\{[\s\S]*?grid-template-columns:\s*160px/)
expect(settingsView).toMatch(/\.settings-view\s*\{[\s\S]*?padding:\s*5px/)
expect(settingsView).toMatch(/\.options-section-button\s*\{[\s\S]*?min-height:\s*16px/)
it('keeps Options tree/content chrome at capture CSS scale and wired section ids', () => {
expect(css).toMatch(/\.settings-view\s*\{[\s\S]*?padding:\s*4px 6px/)
expect(css).toMatch(/\.settings-view \.options-section-nav[\s\S]*?border:\s*1px solid #a0a0a0/)
expect(css).toMatch(/\.settings-view \.options-section-button[\s\S]*?min-height:\s*18px/)
expect(css).toMatch(/\.settings-view \.options-tree-group-label[\s\S]*?font-size:\s*11px/)
expect(css).toMatch(/\.settings-view \.n-card-header[\s\S]*?min-height:\s*20px/)

expect(settingsView).toMatch(/data-options-density="capture-1to1"/)
expect(settingsView).toMatch(/\.settings-view\s*\{[\s\S]*?grid-template-columns:\s*176px/)
expect(settingsView).toMatch(/\.settings-view\s*\{[\s\S]*?padding:\s*4px 6px/)
expect(settingsView).toMatch(/\.options-section-button\s*\{[\s\S]*?min-height:\s*18px/)
expect(settingsView).toMatch(/\.options-tree-group-label\s*\{[\s\S]*?font-size:\s*11px/)
expect(settingsView).toMatch(/\.options-hint\s*\{[\s\S]*?line-height:\s*16px/)
expect(settingsView).toMatch(
/\.stack-row input,\s*\.stack-row select\s*\{[\s\S]*?height:\s*20px/,
)
expect(settingsView).toMatch(
/\.stack-row input,\s*\.stack-row select\s*\{[\s\S]*?border:\s*1px solid #a0a0a0/,
)
expect(settingsView).toMatch(/:deep\(\.n-card-header\)\s*\{[\s\S]*?min-height:\s*20px/)
expect(settingsView).not.toMatch(/font-size:\s*9px/)
expect(settingsView).not.toMatch(/min-height:\s*9px/)
expect(settingsView).toMatch(/data-testid="options-content"/)
expect(settingsView).toMatch(/data-testid="options-section-nav"/)
expect(settingsView).toMatch(/options-section-\$\{section\.id\}/)
Expand Down Expand Up @@ -123,9 +138,9 @@ describe('options dialog densify', () => {
expect(settingsView).toMatch(/data-testid="folder-merge-show-center-pane"/)
expect(settingsView).toMatch(/data-testid="folder-merge-compare-to-output"/)
expect(settingsView).toMatch(/\.shortcut-config\s*\{[\s\S]*?gap:\s*4px/)
expect(settingsView).toMatch(/\.shortcut-row\s*\{[\s\S]*?padding:\s*2px 4px/)
expect(settingsView).toMatch(/\.shortcut-row\s*\{[\s\S]*?padding:\s*2px 6px/)
expect(settingsView).toMatch(/\.shortcut-row\s*\{[\s\S]*?border-radius:\s*0/)
expect(settingsView).toMatch(/\.shared-session-list li\s*\{[\s\S]*?padding:\s*2px 4px/)
expect(settingsView).toMatch(/\.shared-session-list li\s*\{[\s\S]*?padding:\s*2px 6px/)
expect(settingsView).toMatch(/\.shared-session-list li\s*\{[\s\S]*?border-radius:\s*0/)
})
})
Expand Down
Loading
Loading