Skip to content

Commit b66fd2a

Browse files
author
DavidQ
committed
Normalize workspace header button sizing and spacing to match toolbars - PR 11.5
Increase PREV/NEXT size and padding - PR 11.6
1 parent 7755a51 commit b66fd2a

11 files changed

Lines changed: 155 additions & 21 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,8 @@
11
model: gpt-5.3-codex
2-
reasoning: medium
32

4-
Apply PR_11_4_FINAL_UI_POLISH_AND_CONSISTENCY_PASS.
3+
Apply PR_11_6_NAV_BUTTON_POLISH
54

6-
- Fix remaining UI inconsistencies
7-
- Normalize borders, spacing, and shadows
8-
- Ensure consistency across all tools
9-
- No behavior/data changes
10-
- Add validation report
5+
- Increase PREV/NEXT size
6+
- Add padding
7+
- Match toolbar buttons
8+
- CSS only

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Final UI polish and consistency pass across all tools - PR 11.4
1+
Increase PREV/NEXT size and padding - PR 11.6
Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,44 @@
1+
# PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION Report
2+
3+
## Result
4+
PASS
5+
6+
## Scope
7+
- Workspace header button normalization to match tool toolbar button sizing/spacing.
8+
- CSS-only adjustments in shared platform shell styles.
9+
- Applied to both normal and fullscreen states.
10+
11+
## Files Changed
12+
- `tools/shared/platformShell.css`
13+
- `docs/dev/reports/PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION_report.md`
14+
15+
## Changes Applied
16+
- Normalized workspace header button scale:
17+
- `.tools-platform-frame__project-button`
18+
- `.tools-platform-frame__link`
19+
- `.tools-platform-frame__action-link`
20+
- `.tools-platform-frame__nav-link`
21+
- Updated to toolbar-aligned sizing:
22+
- `min-height` normalized to `34px` (nav links `32px`)
23+
- `padding` normalized to `6px 12px` (nav links `6px 10px`)
24+
- `border-radius` normalized to `10px` (from pill radius)
25+
- Normalized workspace header action spacing:
26+
- `.tools-platform-frame__project-actions` gap uses shared spacing token
27+
- Fullscreen alignment:
28+
- explicit fullscreen gap normalization for `.tools-platform-frame__project` and `.tools-platform-frame__project-actions`
29+
30+
## Constraint Compliance
31+
- No behavior changes.
32+
- No data/schema/sample/loader changes.
33+
- No `start_of_day` changes.
34+
35+
## Validation
36+
Command run:
37+
- `node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --tools`
38+
39+
Result:
40+
- PASS `18/18`
41+
- FAIL `0`
42+
43+
## Notes
44+
- Fullscreen header state label text and exit behavior preserved (no JS behavior edits).

docs/dev/reports/REPORT_PR_11_5.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
# REPORT_PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION
2+
3+
Workspace header buttons now match tool toolbar sizing and spacing.

docs/dev/reports/REPORT_PR_11_6.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Nav buttons visually aligned with toolbar.

docs/dev/reports/launch_smoke_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Launch Smoke Report
22

3-
Generated: 2026-04-28T23:10:38.616Z
3+
Generated: 2026-04-28T23:23:15.515Z
44

55
Filters: games=false, samples=false, tools=true, sampleRange=all
66

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# BUILD_PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION
2+
3+
## Required Codex Work
4+
5+
1. Inspect workspace header button styles
6+
2. Compare with tool toolbar button styles
7+
3. Normalize:
8+
- height
9+
- padding
10+
- margin/spacing
11+
- alignment
12+
4. Ensure consistency across:
13+
- normal mode
14+
- fullscreen mode
15+
5. No layout or behavior changes
16+
17+
## Validation
18+
Create:
19+
docs/dev/reports/PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION_report.md
20+
21+
## Constraints
22+
- Visual only
23+
- No logic changes
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# PLAN_PR_11_5_WORKSPACE_HEADER_BUTTON_NORMALIZATION
2+
3+
## Purpose
4+
Normalize workspace header button sizing, spacing, and alignment to match tool toolbar standards.
5+
6+
## Scope
7+
- Workspace header buttons only
8+
- Match size, padding, spacing, and alignment with tool buttons
9+
- No behavior changes
10+
- No data/schema changes
11+
- Do not modify start_of_day folders
12+
13+
## Acceptance
14+
- All header buttons match tool button size
15+
- Consistent spacing between buttons
16+
- Alignment consistent with toolbars
17+
- No visual mismatch remains
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
# BUILD
2+
3+
- Increase button size
4+
- Add padding
5+
- Match toolbar visual weight
6+
- No logic changes
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
# PLAN_PR_11_6_NAV_BUTTON_POLISH
2+
3+
Increase size of [PREV]/[NEXT] buttons and add padding to match toolbar weight.
4+
5+
Scope: visual only.

0 commit comments

Comments
 (0)