ani icon indicating copy to clipboard operation
ani copied to clipboard
verified publisher icon open-ani

编辑评论 UI & 番剧评论

Open StageGuard opened this issue 1 year ago • 12 comments

在视频中展示了所有已经做好的功能

https://github.com/user-attachments/assets/3581a2c1-038f-4c55-918e-6cd262459326

未完成的功能：

  • BGM sticker 选择面板
  • 发送评论到服务器
  • 在 EpisodeViewModel 中对应的 state

StageGuard avatar Jul 24 '24 15:07 StageGuard

某些功能做成縮略會不會比較好 留幾個常用功能在外面就好了 如果留言區只有一小行 顯示起來會不會卡到阿🤔

https://github.com/user-attachments/assets/79983485-6e57-4b62-9df8-fa3012ba384d

NeKoOuO avatar Jul 24 '24 15:07 NeKoOuO

这个分支 luckily 居然没有任何一行 conflict

Him188 avatar Jul 28 '24 18:07 Him188

现在还有点问题：

  • FlowRow horizontal arrangement 能不能 SpaceEvenly 和 Start 一起用
  • 编辑框的光标向上偏移了一个 statusBar 的高度，可能和 Popup 的 popupPositionProvider 有关
  • 编辑框长按没有复制粘贴那个操作栏

StageGuard avatar Jul 30 '24 13:07 StageGuard

  • FlowRow horizontal arrangement 能不能 SpaceEvenly 和 Start 一起用

不能，这逻辑都冲突了

Him188 avatar Jul 30 '24 16:07 Him188

  • FlowRow horizontal arrangement 能不能 SpaceEvenly 和 Start 一起用

不能

那就写自定义layout了

StageGuard avatar Jul 30 '24 16:07 StageGuard

现在的逻辑好像有点问题

StageGuard avatar Jul 31 '24 13:07 StageGuard

逻辑我还没看，我早上只看了一半

Him188 avatar Jul 31 '24 13:07 Him188

逻辑我还没看，我早上只看了一半

可以继续 review

StageGuard avatar Jul 31 '24 16:07 StageGuard

我 expect subject 评论是另一个 PR, 因为这个 PR 要等接口 (which 不知道什么时候才能好) 才能合

Him188 avatar Aug 01 '24 13:08 Him188

我 expect subject 评论是另一个 PR, 因为这个 PR 要等接口 (which 不知道什么时候才能好) 才能合

subject 评论就是那个评分吧，episode 评论还没合并

StageGuard avatar Aug 01 '24 13:08 StageGuard

等 https://github.com/bangumi/server-private/pull/688 合并了再继续

StageGuard avatar Aug 03 '24 14:08 StageGuard

仔细 review 完了

Him188 avatar Aug 04 '24 07:08 Him188

能不能把发评论按钮改成仅在 debug 下显示, 然后 merge 这个

你这个实在是太大了, 一有冲突我就要花数小时看

Him188 avatar Aug 11 '24 17:08 Him188

18 conversations must be resolved before merging.

Him188 avatar Aug 13 '24 17:08 Him188