fix: move sent times below Agent message bubbles

This commit is contained in:
鲨鱼辣椒 committed 2026-10-13 18:07:47 +08:00
1 parent 95e701c4f3
commit 154a21a295
4 files changed
+28 -6

No files matched your search

@@ -8,7 +8,7 @@
- Worktree: /Users/chillishark/Makelore 麦洛/.codex-worktrees/makelore/20261013-chat-message-time-64ebac32
- Base commit: 2899009215b1de8e28b84fc108e880c6e68a6098
- Owner: codex
- Status: Ready for Integration
- Status: Planning
## Scope
@@ -94,3 +94,16 @@
- Loaded new assets/mount verified in the real app. The user's current page had no visible chat message articles, so actual message-time DOM visibility is explicitly pending their natural next chat opening; root did not navigate just to create that evidence. Timestamp semantics/layout/hydration were checked in the120passing focused UI/unit tests and updated E2E assertions; full native E2E not run to preserve desktop focus. Do not claim a real visible-message screenshot check.
- Current runtime now reports Renderer64b4a84/source ownworktree; Main source remains7c51bb4a008e388071ec37156adeae80eff24167. Stage retains the actual old Renderer rollback binding. Application is updated for normal user viewing. Source branch/worktree are retained; no merge/push or cleanup.
- Outcome complete: requested existing-field send/reply timestamps implemented, tested and applied in background. Queue-presentation redesign, wait-duration and the separate upstream replay fix are not included in this time-only result. Earlier exact-finish proposal is superseded by the user's simple message-time request.
## User-Specified Sender Time Placement Correction
- User screenshots show current sender timestamp inside the purple bubble and Codex sender timestamp outside/below its bubble. User explicitly says assistant reply time is correct. Scope: move only sender bubble styling to the message-body and keep the sender timestamp as the existing sibling footer below/right outside the bubble. Preserve purple palette, text, timestamps/tooltip/identity, assistant layout, APIs, and all existing runtime source.
- Same-task bundled check/start/status resumed exact owned branch/worktree/base. Current Renderer64b4a84 matches own committed product, with later95e701c documentation only. Canonical/AGENTS/entry/design context is unchanged and reused; all current peer Scope/Intent/Promotion sections read. No semantic conflict blocks this sender-only CSS correction. Planning Gate: Passed.
- Update mobile padding on the inner bubble, relevant E2E geometry expectations and README. Use headless browser CSS geometry/visual checks without launching visible Electron windows; retain existing timestamp unit regressions and typecheck/build. Stage a fresh immutable overlay against actual64b4 predecessor, then strict quiet/account/input/receipt/CAS checks before background refresh. No new send/copy/edit buttons, assistant time redesign, queue/replay/sourceBackend changes or focus activation.
## Sender Placement Verification Before Refresh
- Only sender selectors in agent-chat.css changed: bubble padding/background/border/radius/shadow moved from the outer article to the existing inner message-body; the existing time sibling is now outside the bubble with8px gap and4px right inset. Mobile body padding follows the same split. No sender/assistant timestamp values, labels, message logic or assistant selectors changed. README and Electron spec now reflect the inner bubble and assert footer below/outside it.
- Headless Chromium QA with real MessageTimestamp React SSR, bundled styles/fonts and synthetic short/long/unbroken messages passed at320/375/980/1366/1920. Sender wrapper/footer transparent, purple bubble retained,8px gap/4px text inset, wrapping correct, no horizontal overflow. Assistant bubble/time computed styles and dimensions match oldHEAD baseline. Root reviewed the1366 screenshot; child reviewed320/1366. No real app or network/private data touched by this visual check. Evidence: /Users/chillishark/Makelore 麦洛/.trial-runtime/chat-time-placement-20261013-64ebac32/qa/geometry.json andplacement-*.png.
- Existing43focused timestamp/rendering/reasoning/stream/formatter tests passed, typecheck/production build/scoped E2E lint/diff checks pass. No new implementation-mirroring unit tests added for this CSS-only correction; the updated browser geometry assertion and actual headless layout checks verify the requested behavior. Native Electron E2E is not launched to preserve user focus.
- Fresh separate deployment stage /Users/chillishark/Makelore 麦洛/.trial-runtime/chat-time-placement-20261013-64ebac32 retains reviewed prior helper guards and restricts runtime source delta toagent-chat.css against64b4a84. Main/shared/backend remain unchanged. Sender geometry will also be checked if real messages are visible during post-refresh read-only verification.