Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support inbox message #322

Merged
merged 25 commits into from
May 30, 2024
Merged

Support inbox message #322

merged 25 commits into from
May 30, 2024

Conversation

dragonflylee
Copy link
Contributor

@dragonflylee dragonflylee commented Apr 27, 2024

实现 #197 中提到的邮件功能

  • 聊天列表获取
  • @消息
  • 点赞消息
  • 回复消息
  • 聊天消息暂未实现实时刷新
  • 点击回复消息跳转到视频页

屏幕截图 2024-04-27 152737

屏幕截图 2024-04-27 152417

@dragonflylee dragonflylee changed the title Support inbox message [WIP] Support inbox message Apr 27, 2024
@dragonflylee dragonflylee force-pushed the inbox branch 2 times, most recently from 9086b5c to fc995c5 Compare May 1, 2024 11:55
@dragonflylee dragonflylee force-pushed the inbox branch 2 times, most recently from c38a175 to c581822 Compare May 7, 2024 13:11
@dragonflylee dragonflylee changed the title [WIP] Support inbox message Support inbox message May 7, 2024
@dragonflylee
Copy link
Contributor Author

第一版只能手动刷新消息,实时推送 暂时没搞清楚怎么弄

@dragonflylee dragonflylee force-pushed the inbox branch 5 times, most recently from 3a79eaf to b1b7bad Compare May 10, 2024 12:40
@xfangfang
Copy link
Owner

xfangfang commented May 20, 2024

抱歉这么久才开始重新关注回这个PR,真的实现的相当完美了,下面我稍微吹毛求疵一点,如果最近时间不充裕的话我可以先合并进来然后我来搞一下:

  1. 有部分引用的字符串没有写在i18n的json里
image image image
  1. 字符串过长时显示的位置出现偏差

示例1:
image

image

示例2:
image

  1. 聊天对话框和刷新按钮有交叉
image

感觉这种情况可以调整一下左侧聊天框的最大尺寸?或者调整一下刷新框的位置或大小

  1. 在聊天列表页按 x,应该是有页面刷新的效果的?但是右下角的刷新按钮没有旋转。而且焦点会有些跳变,我感觉这种情况可以在按x之后把焦点和列表滑动的位置调整到顶部?

  2. 聊天列表初始化的滑动位置可能有些小问题

这个聊天页面有两条对话,但是刚刚进入时长这样:
image

焦点切换后:
image

(顺便一提,感觉时间和焦点框离得有点近了)

  1. 我还是感觉这里聊天应该在设置上面(和官方保持一致)
image

@dragonflylee
Copy link
Contributor Author

dragonflylee commented May 21, 2024

修改了 1. 中的翻译问题
增加了聊天消息中的时间和焦点框离
主界面聊天按钮的位置调整为设置按钮上方

@xfangfang
Copy link
Owner

修复或者调整了一些和尺寸相关的小问题,将上面 2 中各种字符串过长时候的位置问题都调整好了。

顺便发现了一个新的问题:
7. 列表没有做自动加载,比如如果收到过很多不同的点赞,只有第一页会显示,拉到下面不会加载下一页。

@xfangfang
Copy link
Owner

目前还没有解决的问题有:
5. 聊天列表初始化的滑动位置可能有些小问题
7. 列表没有做自动加载下一页,比如如果收到过很多不同的点赞,只有第一页会显示,拉到下面不会加载下一页。
8. 聊天页没有自动加载下一页(上一页),只能看到最近20条

7/8 感觉暂时不做也行,5 本质是列表的问题,这个以后有时间再弄吧。

没什么其他的修改,我就先合并啦~

@dragonflylee
Copy link
Contributor Author

dragonflylee commented May 29, 2024

  1. 列表没有做自动加载下一页 是因为我这个账户没有这么多赞,没抓到相应的报文。接口已经预留了 MsgFeedCursor 作为翻页依据

@xfangfang
Copy link
Owner

7 把回复/点赞/@ 这三个页面的自动加载完成了(聊天列表的自动加载还没做,感觉应该很少有人会在B站聊很多天,有个最近聊天已经足够了)

@xfangfang xfangfang merged commit ab79071 into xfangfang:dev May 30, 2024
16 checks passed
@dragonflylee dragonflylee deleted the inbox branch May 30, 2024 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants