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

🔄 项目重构 #174

Open
11 of 15 tasks
imsyy opened this issue Jun 15, 2024 · 23 comments
Open
11 of 15 tasks

🔄 项目重构 #174

imsyy opened this issue Jun 15, 2024 · 23 comments
Assignees
Labels
重要 Important 重要信息 Important information 高优先级 P0 这个问题将被高优先级处理 This problem will be dealt with with high priority.

Comments

@imsyy
Copy link
Owner

imsyy commented Jun 15, 2024

没错,又要重构了( 绝对是最后一次 👻 )

最近,在软件的使用过程中,我发现了诸多使用上的不便之处,如一些无需跳转的页面、冗余的功能、歌单页面的分页、杂乱的设置以及大量屎山代码等。于是决定再进行一次重构,为了加快重构进程,除非遇到重大问题导致软件无法正常使用,否则在重构过程中将不会进行积极维护。


目前发现内存溢出问题,定位到或为 Naïve UI 的部分组件及 vue-virtual-scroller 虚拟列表,现造成部分页面卡顿及内存占用过大,计划逐渐替换相关组件

部分开发进展

查看全部

🎵 支持音乐标签编辑

image

🛠 全局设置解耦并重新设计

image

🎨 部分超长标题支持滚动显示

📜 歌单支持自定义排序

image

📜 歌词面板快捷操作

image

@imsyy imsyy self-assigned this Jun 15, 2024
@imsyy imsyy added 高优先级 P0 这个问题将被高优先级处理 This problem will be dealt with with high priority. 重要 Important 重要信息 Important information labels Jun 15, 2024
@imsyy imsyy pinned this issue Jun 15, 2024
@dreamsmaner
Copy link

Win10下运行使用内存300多,对比YesPlayMusic是200多,但使用体验我觉得Splayer更舒服,加油。

@swfeng00
Copy link

太勤勉了

@ui-beam-9
Copy link

加油!

@Ks9hwY8
Copy link

Ks9hwY8 commented Jul 8, 2024

You're awesome, come on man

@SlimeNull
Copy link

Win10下运行使用内存300多,对比YesPlayMusic是200多,但使用体验我觉得Splayer更舒服,加油。

噗, 我这 SPlayer 吃了五六百兆的内存... 程序的分辨率是 1500x870

image

@SlimeNull
Copy link

而且在播放页, 核显占用更恐怖 )
image

@imsyy
Copy link
Owner Author

imsyy commented Jul 17, 2024

而且在播放页, 核显占用更恐怖 ) image

image
我这跟网易云不相上下😂优化内存这一块实在是不擅长( 毕竟 electron 本身就是一坨 )

@SlimeNull
Copy link

而且在播放页, 核显占用更恐怖 ) image

image 我这跟网易云不相上下😂优化内存这一块实在是不擅长( 毕竟 electron 本身就是一坨 )

我也觉得挺奇怪的. 因为我用调试器删去我觉得占用最大的模糊背景, 歌词之后, 它的占用还是没降下来. 不知道是哪里吃了 GPU

@imsyy
Copy link
Owner Author

imsyy commented Jul 17, 2024

我也觉得挺奇怪的. 因为我用调试器删去我觉得占用最大的模糊背景, 歌词之后, 它的占用还是没降下来. 不知道是哪里吃了 GPU

主要是逐字歌词,目前的版本没有采用 requestAnimationFrame,可能会导致性能问题,可以关了逐字歌词试试( 没有独立显卡不建议开 )。

  • 现版本:
    image

  • 重构版本:(使用 requestAnimationFrame,都开启逐字歌词 )
    image

@SlimeNull
Copy link

原来如此

@akFace
Copy link

akFace commented Aug 3, 2024

能否加个在歌词页面可以拖拽播放器窗口位置 😘

@imsyy
Copy link
Owner Author

imsyy commented Aug 5, 2024

能否加个在歌词页面可以拖拽播放器窗口位置 😘

ok

@WuChunyang97
Copy link

加油

@yangtian997

This comment was marked as spam.

@akFace
Copy link

akFace commented Sep 13, 2024

什么时候来个重构后的体验版尝尝鲜,等了好久了都 🥰

@imsyy
Copy link
Owner Author

imsyy commented Sep 14, 2024

什么时候来个重构后的体验版尝尝鲜,等了好久了都 🥰

快了快了👻

@akFace
Copy link

akFace commented Sep 15, 2024

什么时候来个重构后的体验版尝尝鲜,等了好久了都 🥰

快了快了👻

good!

imsyy added a commit that referenced this issue Sep 26, 2024
@imsyy
Copy link
Owner Author

imsyy commented Sep 26, 2024

什么时候来个重构后的体验版尝尝鲜,等了好久了都 🥰

快了快了👻

good!

@akFace 可以尝试下,https://github.com/imsyy/SPlayer/actions/runs/11045292661 ,需要卸载旧版本

@Starfallan
Copy link

什么时候来个重构后的体验版尝尝鲜,等了好久了都 🥰

快了快了👻

good!

@akFace 可以尝试下,https://github.com/imsyy/SPlayer/actions/runs/11045292661 ,需要卸载旧版本

新版本有个跟隔壁一模一样的误报,卡巴斯基会提示请求使用网络摄像头,好像每个音乐软件都因为这个原因有误报。

由于加入了音频输出设备切换功能,该功能调用了 MediaDevices.enumerateDevices(),可能导致安全软件提示要访问摄像头(目前发现卡巴斯基会提示),直接禁用即可。

看了下之前没有相关issue,是不是在之后正式发包前预防性的加个说明,防止有人来问?

@imsyy
Copy link
Owner Author

imsyy commented Sep 26, 2024

新版本有个跟隔壁一模一样的误报,卡巴斯基会提示请求使用网络摄像头,好像每个音乐软件都因为这个原因有误报。

由于加入了音频输出设备切换功能,该功能调用了 MediaDevices.enumerateDevices(),可能导致安全软件提示要访问摄像头(目前发现卡巴斯基会提示),直接禁用即可。

看了下之前没有相关issue,是不是在之后正式发包前预防性的加个说明,防止有人来问?

确实,已经加了 😂

@subframe7536
Copy link

有兴趣加一个增量更新吗?我写了一个库:https://github.com/subframe7536/electron-incremental-update

@imsyy
Copy link
Owner Author

imsyy commented Oct 15, 2024

有兴趣加一个增量更新吗?我写了一个库:https://github.com/subframe7536/electron-incremental-update

感谢,但是目前我对于 electron 还是不太会,等再学习下吧 😂

@subframe7536
Copy link

有兴趣加一个增量更新吗?我写了一个库:https://github.com/subframe7536/electron-incremental-update

感谢,但是目前我对于 electron 还是不太会,等再学习下吧 😂

我可以提pr(

不过用这个库会把electron-vite换掉,理论上没啥问题,但是我自己也只是做过一个demo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
重要 Important 重要信息 Important information 高优先级 P0 这个问题将被高优先级处理 This problem will be dealt with with high priority.
Projects
Status: In Progress
Development

No branches or pull requests