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

[BUG] metingJs 引入音乐后 目录链接失效 #292

Closed
geekswg opened this issue Feb 26, 2023 · 6 comments
Closed

[BUG] metingJs 引入音乐后 目录链接失效 #292

geekswg opened this issue Feb 26, 2023 · 6 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@geekswg
Copy link

geekswg commented Feb 26, 2023

Describe the bug 描述你遇到的错误

使用dev分支主题。
在自定义aside.html 中引入 meting音乐后,文章中的目录链接就失效,点击后没有反应。
注释掉 meting 音乐后,目录恢复正常

Expected behavior 期待的行为

No response

Screenshots 屏幕截图

No response

Build Environment 构建环境

No response

Preview Environment 预览环境

No response

Additional Information 补充信息

No response

@geekswg geekswg added the bug Something isn't working label Feb 26, 2023
@Lruihao

This comment was marked as resolved.

@geekswg geekswg changed the title [BUG] metingJs 引入英语 目录链接失效 [BUG] metingJs 引入音乐后 目录链接失效 Feb 26, 2023
@geekswg
Copy link
Author

geekswg commented Feb 26, 2023

我在本地测试好像没有报错,目录就是点了 没有跳转,地址变了。
注释掉 meting 音乐后,目录链接正常。
aside 添加的代码如下

<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/aplayer/dist/APlayer.min.css">
<script src="https://cdn.jsdelivr.net/npm/aplayer/dist/APlayer.min.js"></script>
<!-- require MetingJS -->
<script src="https://cdn.jsdelivr.net/npm/meting@2/dist/Meting.min.js"></script>
<meting-js
	server="netease"
	type="playlist" list-folded=true order="random"
	id="6678743774" volume="0.2" list-max-height="340px" >
</meting-js>

@geekswg
Copy link
Author

geekswg commented Mar 2, 2023

看别人说是 Aplayer导致的,请大佬参考
https://blog.wangriyu.wang/2018/06-Aplayer.html

Lruihao added a commit that referenced this issue Mar 3, 2023
@Lruihao Lruihao self-assigned this Mar 3, 2023
@Lruihao Lruihao added this to the v0.2.18 milestone Mar 3, 2023
@Lruihao Lruihao moved this from To do to Done in Roadmap of FixIt Mar 3, 2023
@Lruihao
Copy link
Member

Lruihao commented Mar 3, 2023

已处理

@Lruihao Lruihao closed this as completed Mar 3, 2023
@geekswg
Copy link
Author

geekswg commented Mar 3, 2023

请问 v0.2.18 发布了吗,我现在为什么使用mater或者dev版本编译报错,切到0.2.17,才能正常启动。
image
请问改了什么配置吗

@Lruihao Lruihao modified the milestones: v0.2.18-lts, v0.3.x Jul 22, 2024
@Lruihao Lruihao reopened this Jul 22, 2024
@github-project-automation github-project-automation bot moved this from Done to In progress in Roadmap of FixIt Jul 22, 2024
@Lruihao
Copy link
Member

Lruihao commented Jul 22, 2024

中文内容标题后的链接仍然存在bug,如之前讨论一样,这是下游仓库造成的,正确解决方式应该由 alicelieutier/smoothScroll#49 处理,而不是 DIYgod/APlayer#217 ,但是我仍然觉得 APlayer 有必要在它这层阻断这个 BUG,因为我觉得即使 APlayer 需要 smooth scroll,应该限定于它自己的 DOM,而不是影响到页面上其他的锚点滚动行为,这很侵入...


Anyway,既然这个 BUG 表现于 FixIt 主题,那么我想我应该在主题代码中做些什么 dirty hack,所以我先手动移除了 smoothScroll 绑定在 标题和目录链接上的点击事件来避免这个问题...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants