Kazumi icon indicating copy to clipboard operation
Kazumi copied to clipboard

[fixed]: 增加了对评论区固定功能的支持

Open aliferne opened this issue 1 month ago • 5 comments

相关 Issue

#1304

额……这个貌似和那个 #26 的 pr 混在一起了,如果那个 pr 能过的话这个应该可以一并过,不然可能就又要 cherrypick 了

aliferne avatar Nov 09 '25 07:11 aliferne

我对这个PR的实现有些疑问,因为我没有记错的话,我们曾经在类似的场合不止一次地错误使用了 AutomaticKeepAliveClientMixin 并搞砸了一些事情

我想知道 @ErBWs 怎么看这个实现

此外尽量为不同的PR单独建立分支,而不是从单一的开发分支进行PR,这会让PR混在一起

Predidit avatar Nov 09 '25 10:11 Predidit

好的,我错误地 merge 了用于 Issue 26 的 PR,污染了这个分支 我测试下来并没什么问题,不太清楚测试效果如何 如果 AutomaticKeepAliveClientMixin 不稳定,或许可以考虑为 ScrollController 加入监听函数

aliferne avatar Nov 09 '25 10:11 aliferne

我们需要等待 @ErBWs 的意见,他是你修改的这部分代码的所有者,并且我们最开始搞砸了

Predidit avatar Nov 09 '25 10:11 Predidit

这个方案应该是行不通的,见 https://github.com/Predidit/Kazumi/pull/689

不过这个代码看着和我当时写的不完全一样,需要做更多设备上的测试,也许是我当时写的有问题

ErBWs avatar Nov 09 '25 10:11 ErBWs

遗憾,我会等待更多的测试用例的结果

aliferne avatar Nov 09 '25 10:11 aliferne