Skip to content

改变一些东西 #2

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

Open
wants to merge 1 commit into
base: 1.0.0
Choose a base branch
from
Open

改变一些东西 #2

wants to merge 1 commit into from

Conversation

gzyhelloworld
Copy link

致谢,通过你的项目学到了很多东西,看了一遍代码后
我对代码做出了一些调整

  1. 页面样式做出了修改
  2. 函数做出了修改,因为切换歌曲的功能跟下一曲上一曲功能是一样的所以把它弄成一个函数
  3. bug修复,删除歌曲那里有bug,在删除歌曲时this.index没变,最终会导致删除别的曲目时实际上是删除这个的
  4. 修改了滚动条样式

ax

@microzz
Copy link
Owner

microzz commented Apr 6, 2017

@gzyhelloworld 能够相互学习非常开心。1.0.0版本的播放器也是我刚学Vue一两天上手的demo,很多代码现在看来是非常不好的,所以我重新编写代码做了一个2.0.0版本,现在已经差不多做完了,最新代码估计最近两天上传,上传之后我会通知你,希望提出宝贵意见。新版使用Vue+Vuex+vue-router+axios,加入音乐在线搜索功能,加入韩寒的One(一个)等等😄

@microzz
Copy link
Owner

microzz commented Apr 8, 2017

@gzyhelloworld 升级版2.0.0完成了,请切换到2.0.0分支,详看👉https://microzz.com/2017/04/08/vue-music2/

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.

2 participants