Skip to content

feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播#5886

Merged
HenryQW merged 13 commits intoDIYgod:masterfrom
shunf4:feat-more-readable-social
Oct 18, 2020
Merged

feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播#5886
HenryQW merged 13 commits intoDIYgod:masterfrom
shunf4:feat-more-readable-social

Conversation

@shunf4
Copy link
Copy Markdown
Contributor

@shunf4 shunf4 commented Oct 14, 2020

目的 / What this PR does

  • 更可读的微博个人时间线、Twitter 用户时间线、豆瓣用户广播
  • 可能修复了 bug
  • 潜在地引入了新的 bug

更可读的豆瓣读书 RSS

更可读的 DIYgod Twitter RSS

更可读的微博小秘书 微博 RSS

该 PR 相关 Issue / Involved issue

完整路由地址 / Example for the proposed route(s)

https://rsshub.app/douban/people/113894409/status
https://rsshub.app/weibo/user/1642909335
https://rsshub.app/twitter/user/diygod

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 14, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/diy/rsshub/ps7o35nk6
✅ Preview: https://rsshub-git-feat-more-readable-social.diy.vercel.app

[ci skip] [skip ci]
@kasora
Copy link
Copy Markdown
Contributor

kasora commented Oct 18, 2020

这个修改似乎在 twitter user 的推送中将用户头像添加到了 description 中. 请问是 bug 还是 feature. 因为这个行为和其他的推送都不同.

@ruleroller
Copy link
Copy Markdown
Contributor

新浪微博也有这样的情况,有点僭越了,建议在title和description里拿掉用户头像和发博(推)人的昵称。

5a2e252#commitcomment-43340424

@ruleroller
Copy link
Copy Markdown
Contributor

@shunf4 老哥,你还把Twitter里utils的 Re: -> 🔁 ,文字变成了符号,这样不想看到订阅者的retweet的推文的人,之前只要 filterout=Re:就行了,现在弄得更麻烦了,我也没试过图标能否正确过滤。这样的美化工作建议都自己本地操作。 @HenryQW ,建议revert相关的pr。

@Rongronggg9
Copy link
Copy Markdown
Contributor

Rongronggg9 commented Oct 18, 2020

新浪微博也有这样的情况,有点僭越了,建议在title和description里拿掉用户头像和发博(推)人的昵称。

5a2e252#commitcomment-43340424

同意,确实僭越,RSS 属性里已经有博主名字和头像,再加在 item 下的 title 和 description 完全多此一举。
要达到作者理想中的效果,除非阅读器只显示 item 的 description 且不以任何形式显示 RSS 属性,甚至不是以类似文章的形式展示 RSS,而是让 RSS 完全时间线化,这是舍本逐末的。

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 18, 2020

确实,我提这个PR只是作功能建议,所有改动都放在上面图片内,看开发者和社区是否接受,现在看来还需要进一步的修改/加控制参数/revert

大家看哪种好

@shunf4 老哥,你还把Twitter里utils的 Re: -> 🔁 ,文字变成了符号,这样不想看到订阅者的retweet的推文的人,之前只要 filterout=Re:就行了,现在弄得更麻烦了,我也没试过图标能否正确过滤。这样的美化工作建议都自己本地操作。 @HenryQW ,建议revert相关的pr。

@ruleroller 我记得有一个参数可以直接过滤掉retweet

@Rongronggg9
Copy link
Copy Markdown
Contributor

Rongronggg9 commented Oct 18, 2020

现在看来还需要进一步的修改/加控制参数/revert

在实现参数控制前,我认为尽快 revert 恢复原状会比较好,现在的输出并不符合许多使用者乃至阅读器对「RSS 的 description 应该是正文」的预期。

@ruleroller
Copy link
Copy Markdown
Contributor

@ruleroller 我记得有一个参数可以直接过滤掉retweet

刚去查询了文档,是有相关的参数,应该是后来增加的。但还是建议用恢复原来的文字形式,首先确保不会让和我一样用filterout=Re: 过滤的人在更新后发现过滤失败,其次,诚然emoji图片更直观和美观,可不能保证在任一阅读器里不会显示成乱码,且该emoji能显示在每个客户端表现形式也不统一。

我自己其实本地也做了许多的类似的美化,但深知别人不一定和我有一致的喜好,所以从来自用。如果不是功能上出现的bug,已有的路由的美化,还是只在本地做为好。

@Rongronggg9
Copy link
Copy Markdown
Contributor

另外,这个 PR 改变了图片的输出,给图片添加了到原图的超链接,这也是不太有必要的。
阅读器多种多样,行为不尽相同,我们不能排除这么一个可能:点击图片后,阅读器打开了浏览器访问超链接,而不是使用内建的大图预览,那可是很让人抓狂的事。

@ruleroller
Copy link
Copy Markdown
Contributor

ruleroller commented Oct 18, 2020

所以,我提议请 @HenryQW @DIYgod 先尽快revert一下,待 @shunf4 妥善处置后重新提交合适的pr。(或者不提,233)

@ruleroller
Copy link
Copy Markdown
Contributor

说话间,又有人提相关的 issue 了。 #5927

@mashirozx
Copy link
Copy Markdown
Contributor

一点小小的建议:这个形式也有道理,但是并非每个用户都需要,建议做成一个选项,或者新增一个路由。

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 18, 2020

我最近没什么时间,还是revert好,造成麻烦十分抱歉。

不过豆瓣广播原来的正文少了很多内容(主要是转发只显示原文)。待revert后我提个issue

@MisteryMonster
Copy link
Copy Markdown
Contributor

其实不至于 Revert ,很多改进我是支持的,比如转发的内容是更加完整的👏,在转发上用 /blockquote 区分更美观👍,而且豆瓣竟然能看到回复(不过不太可能做到实时更新体验,建议去掉😥)另外那个九宫格我在 TTRSS 上看不太到,虽然很可惜但是看图方式和之前没什么差别。(用的是什么reader 啊)

我觉得大家可以讨论一下是不是所有改动都不值得保留🤔。

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 18, 2020

@MisteryMonster 如果内容有变更,即使是加一个HTML节点这种变化都有可能让一些人的XPath崩掉。这个还是要权衡吧,看社区怎么反馈。我建议如果要保留那就以配置项的方式开启,默认还是和原来一样

我的阅读器是FreshRSS。图片大小和排版这种东西可能更依赖于阅读器,毕竟比如SimplePie这种库会过滤掉内联的style(?)

@mashirozx
Copy link
Copy Markdown
Contributor

建议在我们讨论和提出完善的解决方案前先revert吧,现在已经用不了了 (TдT)

@Rongronggg9
Copy link
Copy Markdown
Contributor

Rongronggg9 commented Oct 18, 2020

我觉得大家可以讨论一下是不是所有改动都不值得保留🤔。

问题不在于改动值不值得保留,而是在于其直接大幅改动了默认输出,这导致了一连串的问题。
改动值得以当前形式保留,当且仅当其不对默认输出造成根本性改变,否则还是应该以参数的方式开启新输出。

@MisteryMonster
Copy link
Copy Markdown
Contributor

也是,大家不能用的确实比好看更加紧急,先 revert 。我们去 telegram 里面讨论吧?

@shunf4
Copy link
Copy Markdown
Contributor Author

shunf4 commented Oct 26, 2020

各位好,#6004 加入了 query string 开启可读性的选项,默认情况下不会有太大改动。

期待各位的讨论和反馈

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.

8 participants