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

修复 javabus.com 的链接在新标签页打开的问题 #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

修复 javabus.com 的链接在新标签页打开的问题 #29

wants to merge 2 commits into from

Conversation

leyle
Copy link

@leyle leyle commented Oct 4, 2020

javbus.com 网站的数据被请求后,链接并没有添加上 target=_blank,故做了两点修改:
1、在 fetch 数据 parse 了 html 后,给列表页的 item 的 a tag 添加上 target 属性;
2、给非列表页的链接添加 target 属性。

@hobbyfang
Copy link
Owner

这个影响了我的脚本其他功能,冲突了,目前看来只能瀑布流那段可以。在线接收部分代码我还没搞懂怎么操作,等我下次提交代码放入吧。

@leyle
Copy link
Author

leyle commented Oct 8, 2020

这个影响了我的脚本其他功能,冲突了,目前看来只能瀑布流那段可以。在线接收部分代码我还没搞懂怎么操作,等我下次提交代码放入吧。

嗯,给所有链接加 target 这个功能,我发现影响范围太大了,后面我改了下,用了个丑陋实现,限制到仅 javbus 网站生效。
大佬你看看怎么实现优雅,期待更新。

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