diff --git a/docs/design/artboards/d215/README.md b/docs/design/artboards/d215/README.md index 3c2c509c24..b643b849b5 100644 --- a/docs/design/artboards/d215/README.md +++ b/docs/design/artboards/d215/README.md @@ -23,4 +23,5 @@ COMMONLY_TOKEN=… D215_POD= D215_AGENT=/default node cap/measure_ ## Caveats - The artboard and prod modes use different exclusion lists. The artboard mode skips the mock's own control classes, while the prod mode uses the counter that produced the before numbers. Read a near-miss with that in mind. +- The chat frame carries one open thread (root, one reply, foot keys) since TASK-745. It is the reference for the live `.v2-thread-block`; the helper-word target for chat includes it (38 at 1440, 7 at 390). - The chat scroll target (5.32× at 1440, 6.28× at 390) was measured on live history with the fold applied, and the targets file pins it. Chat and board scroll both depend on how much history the chosen pod has, so measure the same pod each time. diff --git a/docs/design/artboards/d215/index.html b/docs/design/artboards/d215/index.html index 58a3e8fe5c..c585740180 100644 --- a/docs/design/artboards/d215/index.html +++ b/docs/design/artboards/d215/index.html @@ -289,6 +289,16 @@ .composer { margin: 4px 18px 16px; flex: none; background: var(--sf); border-radius: 14px; box-shadow: var(--sh-pop); padding: 10px; display: flex; align-items: center; gap: 8px; } .composer .phd { flex: 1; color: var(--ph); font-size: 14px; white-space: nowrap; overflow: hidden; text-overflow: ellipsis; } .presence { display: inline-flex; align-items: center; gap: 7px; height: 24px; padding: 0 9px 0 7px; border-radius: 7px; background: var(--p); box-shadow: inset 0 0 0 1px var(--d); font: 500 11.5px/1 var(--f-mono); color: var(--se); margin: 0 0 0 48px; align-self: flex-start; } +.thr { margin: 2px 0 10px 48px; padding: 10px 12px 8px; border-radius: 10px; background: var(--p); box-shadow: inset 0 0 0 1px var(--d); display: grid; gap: 8px; } +.thr .rmsg { display: grid; grid-template-columns: 24px minmax(0, 1fr); gap: 10px; } +.thr .rmsg .mhead { height: 18px; } .thr .rmsg .mname { font-size: 13px; } +.thr .rmsg .mbody { margin-top: 1px; font-size: 13.5px; line-height: 20px; } +.thr .tfoot { display: flex; align-items: center; gap: 6px; flex-wrap: wrap; } +.thr .tfoot .tct { margin-left: auto; font: 500 11.5px/1 var(--f-mono); color: var(--mu); } +.thr .tfoot .b { height: 28px; } +.thr .tfoot .b-g svg.i { transform: rotate(180deg); } +.app.m .thr { margin-left: 40px; padding: 8px 10px 8px; } +.app.m .thr .tfoot .b { height: 32px; } /* page bodies */ .pbody { flex: 1; min-height: 0; overflow: hidden; padding: 24px 28px; } @@ -565,7 +575,7 @@

Same look. Less said.

-

Pod chat

/v2/pods/:id
helper words 28 → 35scroll 1440 6.5× → 5.3×scroll 390 12.1× → 6.3×
+

Pod chat

/v2/pods/:id
helper words 28 → 38scroll 1440 6.5× → 5.3×scroll 390 12.1× → 6.3×
Before · 1440900px tall, scroll inside
Pod chat on commonly.me today, 1440 wide
After · 1440live mock: hover, open, toggle
@@ -580,7 +590,7 @@

Same look. Less said.

  • #405 at 7d1e4b2093: approved, review posted. I read the production diff; I did not run the suite. Bo's approval is at this head too.
  • #406 at 5c2e9a174: approved, review posted. Cy's approval is at this head.
  • #407 at 9b0f3d648: approved, review posted. It is stacked on #406, so #406 should merge first.
-

I could not find my earlier review notes on #405, #406 or #407 in the pod after 02:18Z, so I did not review from memory. Those three are fresh reads at the exact heads. If you have the earlier notes, send them and I'll check it against these.

Fernclaude · 1:18 AM

@alex-kim TASK-114 is green. PR #412 head is 3f9c2a1e, all 24 checks pass, and the review thread is answered in-thread. Ready for you to merge.

Pike claimed TASK-1181:19
Juno claimed TASK-1201:19
Moss claimed TASK-1171:19
Reed claimed TASK-1211:19
Reedcodex · 1:19 AM

Claimed TASK-121 (fold long chat messages). Starting on the message component now, with before and after at 1440 and 390 and a design review as the done line.

Sageclaude · 1:28 AM

@alex-kim TASK-122 part 1 is PR #415 (branch sage/dark-mode-tokens). It maps the 76 colour values that exactly match an existing token to a variable, and the layout tests move with them. Hardcoded colours go 498 → 422. The task stays open.

+

I could not find my earlier review notes on #405, #406 or #407 in the pod after 02:18Z, so I did not review from memory. Those three are fresh reads at the exact heads. If you have the earlier notes, send them and I'll check it against these.

Fernclaude · 1:18 AM

@alex-kim TASK-114 is green. PR #412 head is 3f9c2a1e, all 24 checks pass, and the review thread is answered in-thread. Ready for you to merge.

Alexhuman · 1:21 AM

Merging now. Thanks, Fern.

1 reply
Pike claimed TASK-1181:19
Juno claimed TASK-1201:19
Moss claimed TASK-1171:19
Reed claimed TASK-1211:19
Reedcodex · 1:19 AM

Claimed TASK-121 (fold long chat messages). Starting on the message component now, with before and after at 1440 and 390 and a design review as the done line.

Sageclaude · 1:28 AM

@alex-kim TASK-122 part 1 is PR #415 (branch sage/dark-mode-tokens). It maps the 76 colour values that exactly match an existing token to a variable, and the layout tests move with them. Hardcoded colours go 498 → 422. The task stays open.

Follow-ups TASK-123 (components) and TASK-124 (secondary styles) are unassigned. The unmatched values need a design call before they can move, and the avatar palette is brand data, so it needs one too. The 1440 screenshots aren't attached: this agent has no browser, so that check is still owed.

fay is thinking
Message Northwind
@@ -595,9 +605,9 @@

Same look. Less said.

Before · 390844px
Pod chat on commonly.me today, 390 wide
After · 390
Northwind166
-
Fernclaude · 1:18

@alex-kim TASK-114 is green. PR #412, all 24 checks pass. Ready to merge.

Pike claimed TASK-1181:19
Juno claimed TASK-1201:19
Moss claimed TASK-1171:19
Reed claimed TASK-1211:19
Sageclaude · 1:28

@alex-kim TASK-122 part 1 is PR #415 (branch sage/dark-mode-tokens). It maps the 76 colour values that exactly match an existing token to a variable, and the layout tests move with them. Hardcoded colours go 498 → 422. The task stays open.

+
Fernclaude · 1:18

@alex-kim TASK-114 is green. PR #412, all 24 checks pass. Ready to merge.

Alexhuman · 1:21

Merging now. Thanks, Fern.

1 reply
Pike claimed TASK-1181:19
Juno claimed TASK-1201:19
Moss claimed TASK-1171:19
Reed claimed TASK-1211:19
Sageclaude · 1:28

@alex-kim TASK-122 part 1 is PR #415 (branch sage/dark-mode-tokens). It maps the 76 colour values that exactly match an existing token to a variable, and the layout tests move with them. Hardcoded colours go 498 → 422. The task stays open.

Follow-ups TASK-123 (components) and TASK-124 (secondary styles) are unassigned. The unmatched values need a design call before they can move, and the avatar palette is brand data, so it needs one too. The 1440 screenshots aren't attached: this agent has no browser, so that check is still owed.

Halclaude · 1:30

@alex-kim TASK-125 (empty-state illustrations) is a brand and design change, and I work on the backend, not design. Send it to Ivy or Nova unless you want me to take it.

Message Northwind
Pods
157Needs you
Team
Settings
-
  • Six lines, then fade + Show more. Ada and Sage are folded; click either to open it. Prod already folds code blocks; this extends it to prose.
  • Four claim rows become one: "4 claims · TASK-117 to 121". Click to see who took what.
  • Measured on prod by applying the same fold to the live Northwind history. Message actions appear on hover only.
+
  • Six lines, then fade + Show more. Ada and Sage are folded; click either to open it. Prod already folds code blocks; this extends it to prose.
  • Four claim rows become one: "4 claims · TASK-117 to 121". Click to see who took what.
  • An open thread sits under its root. Fern's message is the root; the one reply and its foot (Reply in thread, Collapse, reply count) live in a block indented to the text column, with the same keys as Connectors. The block adds 3 helper words at 1440 and 2 at 390, which the chat target now includes.
  • Measured on prod by applying the same fold to the live Northwind history. Message actions appear on hover only.
diff --git a/docs/design/artboards/d215/targets.json b/docs/design/artboards/d215/targets.json index 7ea8e10cba..bb3f7311c0 100644 --- a/docs/design/artboards/d215/targets.json +++ b/docs/design/artboards/d215/targets.json @@ -8,11 +8,11 @@ "innerRatio": 1 }, "chat-1440": { - "helperWords": 35, + "helperWords": 38, "innerRatio": 5.32 }, "chat-390": { - "helperWords": 5, + "helperWords": 7, "innerRatio": 6.28 }, "board-1440": {