Skip to content

Commit 508dc05

Browse files
author
DavidQ
committed
Make Workspace Manager tile grid wrap responsively - PR 11.32. Fix actual Workspace Manager tile overflow source - PR 11.33
1 parent dde8be8 commit 508dc05

5 files changed

Lines changed: 96 additions & 19 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 28 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,39 @@
11
MODEL: GPT-5.3-codex
2-
REASONING: low
2+
REASONING: medium
33

44
TASK:
5-
Apply PR 11.31.
5+
Apply PR 11.32.
66

7-
Locate the PREV/NEXT navigation label rendering in Workspace Manager.
7+
Fix the Workspace Manager tool tile grid so tiles do not overflow the container when the browser width shrinks.
88

9-
Update styling for the tool name text:
10-
- increase font size (e.g., 1.2x–1.4x current or equivalent px/em bump)
11-
- set color to #3600af
9+
Use responsive wrapping:
10+
- allow cards to wrap to fewer columns
11+
- down to one column on narrow widths
12+
- keep cards within container width
13+
- avoid fixed-width layouts that exceed the panel
14+
- use minmax/min-width: 0/box-sizing as needed
15+
- ensure long labels/status text does not force horizontal overflow
1216

1317
Do NOT:
14-
- change navigation logic
15-
- change payload logic
16-
- change layout containers
17-
- add new components
18+
- change tool data or payload logic
19+
- change palette handoff
20+
- change fullscreen behavior
21+
- change navigation behavior
22+
- touch start_of_day folders
23+
- add broad refactors
24+
25+
Likely targets:
26+
- tools/shared/platformShell.js if styles are inline/generated there
27+
- related shared shell CSS/style block if tile grid styles live elsewhere
1828

1929
Validation:
2030
node --check tools/shared/platformShell.js
31+
node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --samples --sample-range=1902-1902 --tools
2132

22-
Manual:
23-
Open Workspace Manager
24-
Verify PREV/NEXT tool name:
25-
- larger font
26-
- color #3600af
33+
Manual validation:
34+
Open sample 1902 -> Workspace Manager.
35+
Shrink browser width.
36+
Confirm:
37+
- tile cards wrap inside the purple container
38+
- no horizontal runoff
39+
- grid can collapse to one column

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Increase PREV/NEXT label font size and apply color #3600af - PR 11.31
1+
Make Workspace Manager tile grid wrap responsively - PR 11.32
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# PR 11.32 Validation
2+
3+
Changed files:
4+
-
5+
6+
Root cause:
7+
-
8+
9+
Before:
10+
- Workspace Manager tiles overflowed horizontally when browser/container width shrank.
11+
12+
After:
13+
- Tiles wrap within the container and collapse down to one column.
14+
15+
Validation:
16+
- node --check tools/shared/platformShell.js
17+
- node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --samples --sample-range=1902-1902 --tools
18+
19+
Manual:
20+
- sample 1902 -> Workspace Manager -> shrink browser width
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# PR 11.32 — Workspace Tile Grid Responsive Wrap
2+
3+
## Purpose
4+
Fix Workspace Manager tool tiles overflowing horizontally when the browser/container width shrinks.
5+
6+
## Problem
7+
The tool category tiles run past the right edge of the Workspace Manager panel instead of wrapping within the available container width.
8+
9+
## Required Change
10+
Update the Workspace Manager tile grid layout so tiles wrap responsively down to one column.
11+
12+
## Scope
13+
- Layout/CSS/rendering only for Workspace Manager tile grids.
14+
- Do not change tool data, payload logic, palette handoff, fullscreen behavior, or navigation behavior.
15+
- Do not touch start_of_day folders.
16+
17+
## Acceptance
18+
- Tool tiles stay inside the Workspace Manager container at narrow widths.
19+
- Grid wraps from multiple columns down to one column.
20+
- No horizontal overflow from tile cards.
21+
- Existing tool labels/status pills remain readable.
22+
- Runtime smoke test passes.

tools/shared/platformShell.css

Lines changed: 25 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -199,11 +199,13 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
199199
border-radius: 12px;
200200
background: linear-gradient(180deg, var(--panel, rgba(76, 29, 149, 0.88)) 0%, var(--panel2, rgba(49, 17, 102, 0.94)) 100%);
201201
box-shadow: var(--tools-shell-shadow);
202-
min-width: 220px;
202+
min-width: 0;
203203
min-height: 120px;
204204
height: auto;
205205
align-self: stretch;
206206
justify-content: flex-start;
207+
max-width: 100%;
208+
box-sizing: border-box;
207209
}
208210

209211
.tools-platform-frame__nav-bucket-title {
@@ -221,6 +223,7 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
221223
gap: 6px;
222224
flex-wrap: nowrap;
223225
flex-direction: column;
226+
min-width: 0;
224227
}
225228

226229
.tools-platform-frame__nav-tool-row {
@@ -231,6 +234,7 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
231234
padding-bottom: 4px;
232235
margin-bottom: 4px;
233236
border-bottom: 0;
237+
min-width: 0;
234238
}
235239

236240
.tools-platform-frame__nav-tool-row:last-child {
@@ -249,6 +253,8 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
249253
gap: 6px;
250254
margin: 0;
251255
pointer-events: none;
256+
min-width: 0;
257+
max-width: 100%;
252258
}
253259

254260
.tools-platform-frame__binding-badge {
@@ -262,6 +268,10 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
262268
color: var(--muted, #e9ddff);
263269
background: rgba(43, 16, 91, 0.75);
264270
pointer-events: none;
271+
max-width: 100%;
272+
white-space: nowrap;
273+
overflow: hidden;
274+
text-overflow: ellipsis;
265275
}
266276

267277
.tools-platform-frame__binding-badge.is-active {
@@ -280,13 +290,16 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
280290

281291
.tools-platform-frame__nav {
282292
display: grid;
283-
grid-template-columns: repeat(auto-fit, minmax(240px, 1fr));
293+
grid-template-columns: repeat(auto-fit, minmax(min(100%, 240px), 1fr));
284294
grid-auto-rows: auto;
285295
gap: 12px;
286296
align-items: start;
287297
align-content: start;
288298
justify-items: stretch;
289299
width: 100%;
300+
min-width: 0;
301+
overflow-x: hidden;
302+
box-sizing: border-box;
290303
}
291304

292305
.tools-platform-frame__nav.is-workspace-locked {
@@ -340,6 +353,7 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
340353
.tools-platform-frame__nav-section {
341354
display: grid;
342355
gap: 10px;
356+
min-width: 0;
343357
}
344358

345359
.tools-platform-frame__nav > .tools-platform-frame__nav-section,
@@ -357,13 +371,16 @@ body.tools-platform-workspace-context .tools-platform-frame__description {
357371

358372
.tools-platform-frame__nav-grid {
359373
display: grid;
360-
grid-template-columns: repeat(auto-fit, minmax(240px, 1fr));
374+
grid-template-columns: repeat(auto-fit, minmax(min(100%, 240px), 1fr));
361375
grid-auto-rows: auto;
362376
gap: 12px;
363377
align-items: start;
364378
align-content: start;
365379
justify-items: stretch;
366380
width: 100%;
381+
min-width: 0;
382+
overflow-x: hidden;
383+
box-sizing: border-box;
367384
}
368385

369386
.tools-platform-frame__divider--nav-split {
@@ -503,6 +520,11 @@ body.tools-platform-workspace-context .tools-platform-frame__shared-status stron
503520
padding: 6px 10px;
504521
color: var(--muted, #e9ddff);
505522
font-size: 0.88rem;
523+
min-width: 0;
524+
max-width: 100%;
525+
overflow: hidden;
526+
text-overflow: ellipsis;
527+
white-space: nowrap;
506528
}
507529

508530
.tools-platform-frame__nav-link.is-disabled {

0 commit comments

Comments
 (0)