Merged
Conversation
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
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.
背景
Threads 顶栏这轮有三个明显问题:
first captured today,导致文案很快进入省略。Search conversations挤成类似conversationr的错误截断观感。本次修正
first captured today过早进入省略。Cancel降为更弱的次级动作Search conversationr这类裁断伪像非目标
验证
frontend\\node_modules\\.bin\\tsc.cmd -p frontend\\tsconfig.json --noEmitpnpm -C frontend buildpnpm -C vesti-web build手测重点
first captured today的可读长度明显更长,不再只剩todCancel不再突兀,明显是次级动作Search conversations空态显示完整,不再出现conversationr