-
Notifications
You must be signed in to change notification settings - Fork 618
fix: Aplayer显示问题 #939
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
fix: Aplayer显示问题 #939
Conversation
|
再次感谢您对 Volantis 的支持! 此功能涉及到了功能模块,需要等待其他开发人员进行测试。 查看代码,逻辑正确,如果没有其他维护人员测试通过,我会在 5 月 31 号直接通过合并。 非常感谢你的支持! |
发现了些错误,我先变为草稿 |
|
刚刚在进一步尝试,发现这么改仍然会报错。看错误信息,是layout中的 |
|
周未还在改代码,太强了,很晚了,先休息吧,祝你明天有个愉快的周未 |
|
好噢,感谢噢 |
|
@Tikas 刚才测试了一下,music字段可以被传递进去,但是后来发现page页面没有把layout的给插上去导致出现了两个音乐播放器,在layout加了个page的判断就正常力。 |
|
....! 我是七点多被台胞问相关问题,才起来的,没想到看着源码(其他项目),竟然能看到你的回复,好吧,你这也太早了吧! 不多说了,快出去玩耍,享受周未吧! |
|
😹8点半了还早嘛? |
|
在上方一顿更改下,还是会报错,只是不影响渲染,但是这样对于使用者来说,十分不友好,于是让ai修改了一下aplayer的layout的判断源码,改成这样之后就能检查到了。 刚才在写文章,恰好看到了页面渲染多一个mini播放器,于是我就在想能不能解决掉。 |
|
这么多问题,看来 7 天后不能直接合并,此修改需要进行测试后,才能合并 |
没事,我慢慢测试)。 |
其实现在还剩个问题是适配于page和post页面的aplayer似乎无法被右键菜单给识别到,不过对于这个我并不是很了解,所以我在这留下个疑问(?) |
|
原来因为判断了page和post导致在无music字段的时候原有aplayer没注入到里面去,现在加个判断正常注入应该就行了。🤔 |
|
@Tikas 修改完了,在学校深思了一下,发现最终要改的只有这一个文件,在经过多次本地测试之后,现在应该没问题了) |
|
太强了,这个涉及到了功能模块,我已经告知 xaoxuu ,看看他们怎么检查测试。 |
PR Type
Description
Aplayer有Fixed模式,但是在输入
Fixed: true后,发现预想中的吸底模式并没有出现,在查看主题源文件,得知aplayer下有两个文件,一者是layout.ejs以及script.ejs,两者似乎都会在渲染时插入到html中,理应来说,应该会有个吸底音乐框在下面(此时autoHide: false)。不过实际上并没有,在查看主页的网页源代码的时候,发现
layout.ejs并没有插入到html中,只有script.ejs插入了。经过一个偶然的事情,Hexo控制台输出了一个渲染报错,位于\themes\volantis\layout\_partial\scripts\index.ejs,这时我才知道,管理插入html的地方在于此处,检查了一下这个文件,发现没有加入layout.ejs,导致了aplayer的吸底模式没有渲染出来,在添加了该行为后,aplayer可以正常渲染ww。查询Hexo官方文档,对于判断是否为Post应该为
is_post()。Others
Issue resolved:
Screenshots with this changes:
Link to demo site with this changes: