refactor(episode_comments_card): 优化剧集评论卡片布局结构#1993
Open
LiggMax wants to merge 3 commits into
Open
Conversation
Contributor
LiggMax
commented
Apr 21, 2026
LiggMax
commented
- 修改为左右内容布局（左侧用户头像，右侧用户昵称、发表时间、评论内容）
- 优化发表时间数据展示效果（使用主题浅灰色、文字缩小）
- 移除回复评论之间的Divider(分割线)，个人觉得缩进的回复内容展示已经可以明确评论之间的回复关系不需要分割线，反而分割线会影响美观
- 修改为左右内容布局（左侧用户头像，右侧用户昵称、发表时间、评论内容） - 优化发表时间数据展示效果（使用主题浅灰色、文字缩小） - 移除回复评论之间的Divider(分割线)，个人觉得缩进的回复内容展示已经可以明确评论之间的回复关系不需要分割线，反而分割线会影响美观
Owner
|
这是负优化吧，屏幕内的信息密度明显变小了，留下了更多的空白 |
- 更新回复内容对其方式 - 添加表面容器最高颜色作为背景色 - 实现圆角边框样式，首尾元素特殊处理
Contributor
Author
这样的效果如何？
|
Owner
|
个人觉得这也不是很好看，我想知道对于这个变更，你的参考是什么，我没有在其他应用上见过类似的设计，将完整的回复卡片用阴影包裹有些奇怪 还是说你只是告诉了人工智能，让它把这里变得好看一点 |
Contributor
Author
代码是手搓的，参考的b站 a站上也是这样设计的 |
Contributor
Author
|
是不是像b站一样默认展示两条回复内容，剩余内容通过点击更多展示会好一些 |
Owner
|
抱歉，我进行了不当的猜测 但是这些设计展示的都是缩略的评论，用阴影包裹完整的回复卡片是很突兀的 当前设计类似 bangumi 网页版，这个 pr 也是设计上的权衡，而不是新的特性或是错误修复。设计上的权衡其实有些微妙，除非有明显的改进，不然不同人的评价标准差距相当大 |
- 评论最多显示2条回复 - 使用嵌套导航器实现评论详情页的局部路由跳转
Contributor
Author
|
这样的改进效果是否好一些 Desktop.2026.04.22.-.01.13.10.03.mp4f1857c018634c12f1def359e870bee0c.mp4 |
Owner
|
这并不够美观，并且和bangumi网页逻辑不符，并且后退手势的处理不恰当 |
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.