fix(ui): designer-audit round 2 — internal jargon, duplicate CTA, doubled explainers - #572
Merged
Conversation
…b split, empty-state placement, vaporware chrome Fixes the P0 batch plus two P1 subtractions from the 2026-07-06 designer audit (15-surface screenshot review): - P0-1 sidebar 会话 label overlapped the 按状态/按项目 toggle on EVERY page: .maka-session-panel declared 4 grid rows for 5 children, so the 1fr track collapsed to 0 and the toggle overflowed onto the list's group label. The toggle's own chrome also referenced tokens that never existed (--surface-secondary/-primary/-tertiary, --foreground-primary, --shadow-sm) and rendered invisible. Fixed with a 5-row template and by replacing the hand-rolled buttons with the shared SettingsSegmented primitive (same family as the daily-review range tabs). - P0-2 sidebar badge said 定时任务 3 while the page tab said 4: the tab badge now counts non-completed reminders like the sidebar does, and the list's default filter is a new 进行中 view (全部/已完成 still selectable) so the number always matches the visible cards. - P0-3 skills 内置 and 已安装 tabs rendered the SAME list — now split by sourceType (bundled vs user-added), with counts on every tab. - P0-4 the shared EmptyState floated at 46% of the page and overlapped banners/cards on module pages: the absolute centering was a sidebar- only recipe living on the base .maka-empty-state class — now scoped to .maka-session-list. Daily review additionally collapses the 无活动 empty card to a one-line note when saved reports already fill the page. - P1-5 skills page now lands on skills the user can run (已安装 → 内置 → market fallback) instead of the all-即将上线 marketplace; removed the 保持系统唤醒·即将支持 tag from the automations banner. - P1-9 removed the dead 全部/排序：热门 pills that were styled like buttons but never wired. Verified per page with CDP screenshots (segmented renders, counts match, built-ins listed, no floating empty card). check-dead-css clean; @maka/ui 45/45; @maka/desktop 2109/2110 — the 1 failure (history compact artifact lifecycle) fails identically on origin/main (pre-existing, flagged separately).
…bled explainers - P1-8: daily-review archive headers printed the raw internal model key (zai-live::glm-4.5). New formatDailyReviewModelLabel shows just the model id — the connection slug is an implementation detail. - P2-14: 通过 Maka 创建 was wired to the EXACT same handler as 新建定时任务 — a duplicate button competing for the primary action. Removed (plus its now-dead CSS and unused Sparkles import); bring a second entry back only when a genuinely different flow exists. - P2-12: every-page double explainer, daily-review edition — the page subtitle now carries the one informative sentence (自动汇总本机对话， 生成摘要、遗漏提醒与深度分析；可在设置中开启定时执行) and the in-card hint paragraph + its CSS are gone. Verified via CDP screenshot: clean model label, single explainer, page reads noticeably calmer. check-dead-css clean; @maka/ui 45/45; @maka/desktop 2109/2110 (1 pre-existing main failure, tracked separately).
jackwener
force-pushed
the
fix/ui-audit-round2
branch
from
July 6, 2026 12:46
c135962 to
4f4102c
Compare
Astro-Han
pushed a commit
that referenced
this pull request
Jul 6, 2026
#557) Adds a status/project view-mode toggle and renders project-grouped sessions as collapsible folder sections with a 4-session preview. Project headers + show-more route through UiButton; collapsed sidebar keeps a 4-row grid; session rows align left under folders. Rebases onto main incl. #571 (SettingsSegmented) and #572.
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
2 participants
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.
设计审查第 2 轮（承接 #571）：
zai-live::glm-4.5）→ 新增formatDailyReviewModelLabel，只显示模型名（glm-4.5）。截图验证通过；check-dead-css ✓；@maka/ui 45/45；desktop 2109/2110（唯一失败为 main 既有 history-compact 用例）。