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

希望增加列表栏预览是否显示图片的选项 #4

Closed
rwbymoe opened this issue May 12, 2022 · 5 comments
Closed

希望增加列表栏预览是否显示图片的选项 #4

rwbymoe opened this issue May 12, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@rwbymoe
Copy link

rwbymoe commented May 12, 2022

无标题

还希望顺便增加评论区头像是否显示的选项,将隐蔽性做到极致

@rwbymoe
Copy link
Author

rwbymoe commented May 13, 2022

还有如果文章里有flash播放器是加载不了的,希望也能去掉

@Tai-Zhou
Copy link
Owner

还有如果文章里有flash播放器是加载不了的,希望也能去掉

请问 flash 是指B站视频链接吗,方便的话能否提供一个链接呢?

@rwbymoe
Copy link
Author

rwbymoe commented May 14, 2022

还有如果文章里有flash播放器是加载不了的,希望也能去掉

请问 flash 是指B站视频链接吗,方便的话能否提供一个链接呢?

https://www.ithome.com/0/618/185.htm
无标题

稍微看了下,newscontent接口里返回的是iframe,里面具体是flash还是video标签应该是由bilibili判断的,你看好不好解决,不好解决就替换成视频已屏蔽的文字好了

@Tai-Zhou Tai-Zhou added the enhancement New feature or request label May 14, 2022
@Tai-Zhou
Copy link
Owner

player-selector.js 中有一段

html5Checker: function () {
    var t = !(/msie [\w.]+/.exec(navigator.userAgent.toLowerCase()) || /Trident/i.test(navigator.userAgent) && /Windows NT 6/.test(navigator.userAgent) || !window.URL);
    if (t) {
        var e = document.createElement("video");
        return e && e.canPlayType && e.canPlayType('video/mp4; codecs="avc1.42001E, mp4a.40.2"')
    }
    return t
}

测试后发现 VSCode 并不支持视频播放,issue 也佐证了这一点。所以B站视频就设置成了显示标题、封面,用链接跳转的形式。

现在新版已经上架啦,您提到的内容应该都实现了。

@rwbymoe
Copy link
Author

rwbymoe commented May 14, 2022

awesome,终于可以在老板眼皮底下看新闻了,给你点个star

@rwbymoe rwbymoe closed this as completed May 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants