-
Notifications
You must be signed in to change notification settings - Fork 40
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
搜索页样式调整 #909
The head ref may contain hidden characters: "\u641C\u7D22\u9875\u6837\u5F0F\u8C03\u6574"
Conversation
这个不能单独摘出来,会破坏缓存策略,试用十月更新最新的commit |
我一刀切缓存跟个commit有很大关系 |
是 |
似乎也没问题,这个事件看你写的逻辑也只会触发一次,还是说关闭页面会导致一些对象不释放? |
测了几遍没发现问题,查看内存也没有看到哪个搜索页中对象没释放的,我用一段时间看看 |
和内存无关,这个commit是半成品,最新的commit才是优化好的搜索页 |
现在来改就有点重复了 |
没什么,我好几天没动过代码了,可能搞混了,d801c26才是破坏的提交 |
奇怪,我测试的时候明明是 |
@ywmoyue 这是一个综合性的问题 总的来说我修改了搜索页里返回上一页的逻辑,让它不再首选视频内容,并加上了缓存,其他的没有变化 |
很失败的试水,还是了解代码未够啊,先优化点UI好了 |
需要继续更新的话,这个才是搜索页的正确样式
|
好的,非常感谢 |
Cherry pick from #898
感谢 @ProJend