【BUG】专辑封面显示不清晰

Ferry-200 / coriander_player

Windows端本地音乐播放器，使用Material You配色。Dart (Flutter) + Rust (lofty, windows-rs) + C (bass lib) 多语言项目。绝赞开发中。

GNU General Public License v3.0

510 stars 22 forks source link

Closed FisherRong closed 1 month ago

FisherRong commented 1 month ago

Bug 概要 专辑封面显示不是很清晰呢

复现步骤 看了很多专辑，都是这样

预期行为

截图

Ferry-200 commented 1 month ago

这是预期行为，故意只加载 400*400 的封面。

FisherRong commented 1 month ago

您好，我已收到

shing-yu commented 1 month ago

不增加点是有什么考虑吗（因为我感觉对于这么大的图来说，400x400有点捉襟见肘（甚至实际占的大小都超过400x400像素点了（

Ferry-200 commented 1 month ago

不增加点是有什么考虑吗（因为我感觉对于这么大的图来说，400x400有点捉襟见肘（甚至实际占的大小都超过400x400像素点了（

有一点内存方面的考虑？虽然没有测试过。

确实我不应该只加载 400400，而是 400 缩放大小。

另外我有一种见解，应该是错误的。就是图片占多大空间就只解码多大的图片。

shing-yu commented 1 month ago

我改800*800测试看看（

Ferry-200 commented 1 month ago

我改800*800测试看看（

我怎么感觉你懂得好多😂

shing-yu commented 1 month ago

我怎么感觉你懂得好多😂

我是Python的开发者，具体可以看主页（虽然我不会dart和flutter，但是编程语言都有相似的地方嘛（而且还有GPT（

Ferry-200 commented 1 month ago

我改800*800测试看看（

如果找不到的话，我这里直接给结果

这里写得确实有点乱。

shing-yu commented 1 month ago

找到了第二个（第一个好像是位置吧（

Ferry-200 commented 1 month ago

找到了第二个（第一个好像是位置吧（

啊对，只用改第二个。改第一个会得到 super large 图片然后填满整个正在播放页（

shing-yu commented 1 month ago

事实上，我改到1200x1200，都没看到cpu和内存的明显提升（所以感觉800x800问题不大（我先pr一下吧（

Ferry-200 commented 1 month ago

事实上，我改到1200x1200，都没看到cpu和内存的明显提升（所以感觉800x800问题不大（我先pr一下吧（

可以，然后后续交流就转到那里。