feat(desktop): refresh Reasonix desktop UI#3816
Conversation
c8a9ad1 to
8349aa1
Compare
8349aa1 to
bb5ffb0
Compare
|
以后涉及ui的变动,最好是附图。纯代码合并风险有些高。 |
|
Stepping back on this one rather than line-reviewing it: a +13k-line shell replacement can't be reviewed or landed as a unit, and several of its choices reverse things that shipped deliberately — it deletes AppChrome/CommandPalette/HistoryPanel/ContextPanel (we merged a palette fix of yours into that exact surface the same day), and swaps the accent identity. #3805 also hasn't had a direction discussion yet; a redesign this size needs sign-off on the proposal before code. The terminal dock additionally needs a real PTY design — the pipe-based session can't host interactive programs, and To be clear, this isn't a "no" to the ideas: VirtualList, useRafThrottle/useStableEvent, the markdown render caching, and the perf scripts would each be welcome as small standalone PRs today — the virtualization piece in particular addresses a known long-session slowdown. And if the #3805 discussion lands on a refresh direction, this branch is the reference implementation for it. Your recent fixes have been excellent; I'd rather take this work in pieces we can actually review than let it rot whole. |
|
Acknowledged. I am leaving this branch as a reference implementation instead of continuing to ask for review as one large shell replacement. First standalone split: #3990, which only virtualizes the existing History/Trash saved-session list and closes #3989. I will keep the broader #3805 redesign in the proposal discussion path, and keep the terminal dock out until there is a real PTY/job-object design. |
Summary
Design Evidence
Validation
npm run check:cssnpm run test:allnpm run typechecknpm run buildnpm run perf:bundlego test ./...fromdesktopgo test ./internal/config -run TestSkillPathMutators