Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

完善评论展示 #175

Merged
merged 2 commits into from
Sep 20, 2021
Merged

完善评论展示 #175

merged 2 commits into from
Sep 20, 2021

Conversation

Richasy
Copy link
Owner

@Richasy Richasy commented Sep 20, 2021

修复 #173 #133 #129

创建一个新的组件,合并回复列表视图与回复详情视图。

PR 类型

这个 PR 的目的是什么?

  • Bug 修复
  • 功能

当前行为是什么?

  1. 播放视频时,点开相关评论的详情,会显示一个居中的弹窗
  2. 点击消息界面的回复信息条目时,会打开浏览器跳转到评论所在的媒体资源页面

新的行为是什么?

  1. 取消打开某一楼层评论详情时的居中弹窗,改为在右侧同位置显示
  2. 支持在回复消息界面直接展开回复详情

PR 检查清单

请检查你的 PR 是否满足以下要求:

  • 应用成功启动
  • 新组件
    • 对于控件,已将控件放在主项目的 Controls 文件夹内
  • 文件头已经被添加至所有源文件中
  • 包含破坏式更新

备注

目前仍不支持跳转到指定的评论处开始浏览

@Richasy Richasy added bug 🐛 异常或错误使得应用不能按预期工作 UI 问题 🎨 用户界面表现不符合预期 操作与体验 🧤 有关操作体验的问题 labels Sep 20, 2021
@Richasy Richasy self-assigned this Sep 20, 2021
@Richasy Richasy linked an issue Sep 20, 2021 that may be closed by this pull request
@Richasy Richasy merged commit 26ac959 into main Sep 20, 2021
@Richasy Richasy deleted the richasy/improveReply branch September 20, 2021 09:39
@Richasy Richasy linked an issue Sep 20, 2021 that may be closed by this pull request
@Degate
Copy link

Degate commented Sep 21, 2021

大佬太拼了,注意休息啊

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug 🐛 异常或错误使得应用不能按预期工作 UI 问题 🎨 用户界面表现不符合预期 操作与体验 🧤 有关操作体验的问题
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]个人消息的处理方式改进建议 [Feature] 评论区全部回复弹出窗口改进建议
2 participants