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

升级程序到v2.0.0 #20

Closed
wants to merge 8 commits into from
Closed

Conversation

zhuzhuyule
Copy link

整理程序代码
修改UI样式
添加快捷键响应
添加下载功能

@listen1
Copy link
Owner

listen1 commented Feb 14, 2018

非常棒的更新,这个PR是对项目非常大的贡献,感谢!

由于这个PR涉及的改动比较多,可能需要几天时间来review,分步并入项目。PR中引入了下载音乐的功能,可能会对主项目造成一定的困扰,我建议暂时在PR中删除这部分的代码,您的个人fork中可以保留这一功能的分支。希望能理解这个项目背负的压力和做出这样选择的初衷。

其中涉及chrome插件版的改动很有价值,主要是解决了主页滚动条的闪动bug,这部分我会直接在chrome_extension项目中merge,并标明代码贡献人。这样桌面版也只需要更新submodule就能够解决这一问题。

Copy link
Owner

@listen1 listen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 部分修改已经整合到chrome_extension库,建议更新submodule(主要是滚动条样式css相关)
  • 移除下载相关代码
  • 加入全局快捷键的说明,能配置就更好了

另外,我能给你fork的代码库提PR吗?好像还没找到怎么弄。

修复自动获取链接有效下载链接
@listen1 listen1 closed this Dec 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants