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

update websites - infoq.cn #2201

Merged
merged 4 commits into from
May 29, 2021
Merged

Conversation

binsee
Copy link
Contributor

@binsee binsee commented May 20, 2021

  • remove invalid rules for infoq.com/cn
  • update infoq.cn

适用于 infoq.cn 全站文章页:

注意事项

由于 InfoQ 站点是异步加载文章内容,因此需要将 infoq.cn 加入到 排除列表。(关于为什么添加到排除列表,而非 延迟加载列表,见 #1074 (comment)
添加到排除列表后,再打开文章页时,按快捷键 A A 进入阅读模式即可。
如未添加到排除列表,因自动进入阅读模式时文章实际内容未加载,会提示错误。

测试用适配源

https://gist.githubusercontent.com/binsee/55acad05b921ba79ac6695a9463f563c/raw/infoq.cn.json

在此pr合并前,可将上述url加入第三方适配源中,打开上述测试页面测试效果。如果进入阅读模式出错,请在站点中选择第三方适配源以启用本适配,而非简悦官方规则中的失效适配
注意pr合并后,请将上述url从第三方适配源中移除,避免后续出问题。

效果演示视频

2021-05-21-HJLxke8pjO.mp4

相关issues

#2035 #2190

- remove invalid rules for infoq.com/cn
- update infoq.cn
Repository owner deleted a comment from hesherya May 21, 2021
@Kenshin
Copy link
Owner

Kenshin commented May 21, 2021

@binsee

抱歉,这几天太忙了,稍晚一阵子我会注意适配它们。另外删了一个不当言论。如给你带来影响,抱歉。谢谢你的贡献。

🙏

1 similar comment
@Kenshin
Copy link
Owner

Kenshin commented May 21, 2021

@binsee

抱歉,这几天太忙了,稍晚一阵子我会注意适配它们。另外删了一个不当言论。如给你带来影响,抱歉。谢谢你的贡献。

🙏

@binsee
Copy link
Contributor Author

binsee commented May 21, 2021

@Kenshin 我通过邮件看到了那条被删除的评论,没关系,不是恶意评论,只是他有个地方没注意到,误以为此适配无效而已。

@hesherya 请注意看图示,启用第三方适配源

  1. 简悦设置 - 站点管理 中,导入适配源
    image

  2. 文章页面中,切换到第三方适配源
    image

  3. 启用第三方适配源后,刷新页面再进入阅读模式

image

@binsee
Copy link
Contributor Author

binsee commented May 21, 2021

我在此pr的顶部comment更新了测试用的适配源url,并增加了效果演示视频,可以更直观的查看了。

github现在能直接添加视频,真是非常给力的功能。

@hesherya
Copy link

@binsee 之前没有用过那个功能,不太了解。您的视频和图示很清晰,我试了一下,很好用,非常感谢。

@Kenshin
Copy link
Owner

Kenshin commented May 21, 2021

嗨,抱歉,今天一直在处理文档的事情,耽搁了。

@Kenshin
Copy link
Owner

Kenshin commented May 21, 2021

关于这个 PR 我后续会 Merge 它,再次感谢 @binsee 的鼎力相助。

@Kenshin Kenshin merged commit a87a5ad into Kenshin:develop May 29, 2021
@Kenshin
Copy link
Owner

Kenshin commented May 29, 2021

嗨,这个站已经 Merge 且已经对外发布。

🙏

@binsee binsee deleted the update-sites-infoq branch May 29, 2021 14:09
@hesherya
Copy link

嗨,这个站已经 Merge 且已经对外发布。

🙏

收到,非常感谢两位。

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

3 participants