gmg137 / netease-cloud-music-gtk

Linux 平台下基于 Rust + GTK 开发的网易云音乐播放器
GNU General Public License v3.0
1.57k stars 95 forks source link

Revert "Custom paintable and background thread" #215

Closed gmg137 closed 1 year ago

gmg137 commented 1 year ago

Reverts gmg137/netease-cloud-music-gtk#203

catsout commented 1 year ago

有什么问题吗?

catsout commented 1 year ago

不难改,之后我给 paintable 加点属性就好

gmg137 commented 1 year ago

我正在尝试看能不能用一种更简单的方法来实现图片加载,虽然使用 paintable 更高级一些,但感觉太复杂了,一眼看不明白背后的逻辑。

catsout commented 1 year ago

随你,我是觉得几百行的代码,算不上复杂,逻辑都封装在 paintable 里,外面不用管。 要改的话,尽量还是都封装起来,外面只用设置 url 就好。 现在要解决 avatar 的这个问题,大概只能用信号,因为 avatar 的 draw_to_texture 方法没法作为 paintable 调用。 给 paintable 加个信号,然后 connect 改 avatar 就行了。