fix(ui): wire the Density toggle into the layout #4

Merged
David merged 2 commits from iterate/make-functional into main 2026-06-05 18:08:02 +02:00
Owner

The Compact/Comfortable control in the status bar and Settings page mutated a Density signal that nothing consumed: Layout read it as _density and never applied a class, and main.css had no density rules, so both controls were no-ops. Add Density::css_class(), apply it on the app root alongside the theme class, and add density rules (declared after the theme blocks so the --vc-row-h override wins by source order) that drive row height and a new --vc-cell-py used by table cell padding.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Signed-off-by: David Randall David@NiceGuyIT.biz

The Compact/Comfortable control in the status bar and Settings page mutated a Density signal that nothing consumed: Layout read it as `_density` and never applied a class, and main.css had no density rules, so both controls were no-ops. Add Density::css_class(), apply it on the app root alongside the theme class, and add density rules (declared after the theme blocks so the --vc-row-h override wins by source order) that drive row height and a new --vc-cell-py used by table cell padding. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: David Randall <David@NiceGuyIT.biz>
fix(ui): wire the Density toggle into the layout
Some checks failed
Check / fmt + clippy + build + tests (pull_request) Has been cancelled
ab289e8a9e
The Compact/Comfortable control in the status bar and Settings page mutated a Density signal that nothing consumed: Layout read it as `_density` and never applied a class, and main.css had no density rules, so both controls were no-ops. Add Density::css_class(), apply it on the app root alongside the theme class, and add density rules (declared after the theme blocks so the --vc-row-h override wins by source order) that drive row height and a new --vc-cell-py used by table cell padding.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: David Randall <David@NiceGuyIT.biz>
Merge branch 'main' into iterate/make-functional
All checks were successful
Create release / Create release from merged PR (pull_request) Has been skipped
Check / fmt + clippy + build + tests (pull_request) Successful in 46s
2893a5aeec
David merged commit 1b97d5f152 into main 2026-06-05 18:08:02 +02:00
David deleted branch iterate/make-functional 2026-06-05 18:08:02 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/vervain-apps!4
No description provided.