fix(ui): reading-column polish — balanced density, conversation radius, flush sticky - #3375
Open
jackwener wants to merge 1 commit into
Open
fix(ui): reading-column polish — balanced density, conversation radius, flush sticky#3375jackwener wants to merge 1 commit into
jackwener wants to merge 1 commit into
Conversation
…s, flush sticky Three cuts verified against a live 8-tool fixture session at 57e08d8 (measured over CDP, before/after screenshots): - ChatMessageList drops the compact+gap={4} override for Astryx's own balanced density. compact with gap={4} was already balanced's gap with compact's padding, so the pin bought nothing but tightness. The real crowding was inside a turn: .maka-processing-sequence gap --space-1 → --space-3 and .maka-assistant-answer-content 8px → 16px take the evidence blocks from a measured 12/8px apart to 20/16px — reading rhythm instead of a log. - User bubble moves from the card tier (10px, read as a settings row) to a new conversation tier: --radius-conversation 16px, single consumer, documented in DESIGN.md §6 as a register exception to the box-height ladder. Assistant stays ghost. - Activity sticky offset --space-2 → 0: the 8px strip above the stuck header let the previous row scroll through half-visible. Flush with the scrollport, both sticky tiers verified (strip measured 0px). Tests: @maka/ui 183 pass, renderer tsc clean, prompt-rail e2e 7 pass.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
对照
docs/astryx-shell-improvement.zh-CN.md/astryx-visual-excellence.zh-CN.md在 57e08d8 上实测（真实 Electron + 8 连续工具调用 fixture，CDP 量测 + 截图）后的结论：文档里 P0.1/P0.2（工具、thinking 默认折叠）与 P1.4（composer 减法）已是现状；.maka-tool-*"第二套 DS ~254 条"的数据已过时（实际 3 个选择器）。真正成立且值得做的三刀在这个 PR 里。改动
1. 阅读列密度（文档 P0.3） —
ChatMessageList去掉compact + gap={4}强制（gap={4} 本来就是 balanced 的 gap，compact 只买到了更紧的 padding）。真正的拥挤在 turn 内部：.maka-processing-sequencegap--space-1 → --space-3、.maka-assistant-answer-content8px → 16px。实测 thinking→tool→正文 的块间距从 12/8px 提到 20/16px。2. 用户气泡 conversation 档（文档 P0.4） — 10px card 档读起来像 settings row。新增
--radius-conversation: 16px（唯一消费者：filled user bubble），DESIGN.md §6 记录为盒高阶梯的 register 例外；assistant 侧保持 ghost 不变。先改了 DESIGN.md 再改 CSS，符合"契约先行"。3. sticky 吸顶齐平 —
--maka-activity-sticky-top从 8px 改 0：8px 偏移在吸顶 header 上方开了一条透缝，上一行滚动时半透出（截图实测）。改 0 后 strip 量测为 0，双层 sticky（组头 + 行头）都正常。验证
@maka/ui183 tests pass；renderertsc --noEmitclean；prompt-raile2e 7 pass明确不做（已核实为非问题）
e2e-fixture.ts设workbarCollapsed: false），真实默认就是收起