feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播#5886
feat: more readable 微博个人时间线, Twitter 用户时间线, 豆瓣用户广播#5886HenryQW merged 13 commits intoDIYgod:masterfrom shunf4:feat-more-readable-social
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/diy/rsshub/ps7o35nk6 |
[ci skip] [skip ci]
…into feat-more-readable-social
[ci skip] [skip ci]
…into feat-more-readable-social
|
这个修改似乎在 twitter user 的推送中将用户头像添加到了 description 中. 请问是 bug 还是 feature. 因为这个行为和其他的推送都不同. |
|
新浪微博也有这样的情况,有点僭越了,建议在title和description里拿掉用户头像和发博(推)人的昵称。 |
同意,确实僭越,RSS 属性里已经有博主名字和头像,再加在 item 下的 title 和 description 完全多此一举。 |
|
确实,我提这个PR只是作功能建议,所有改动都放在上面图片内,看开发者和社区是否接受,现在看来还需要进一步的修改/加控制参数/revert 大家看哪种好
@ruleroller 我记得有一个参数可以直接过滤掉retweet |
在实现参数控制前,我认为尽快 revert 恢复原状会比较好,现在的输出并不符合许多使用者乃至阅读器对「RSS 的 description 应该是正文」的预期。 |
刚去查询了文档,是有相关的参数,应该是后来增加的。但还是建议用恢复原来的文字形式,首先确保不会让和我一样用filterout=Re: 过滤的人在更新后发现过滤失败,其次,诚然emoji图片更直观和美观,可不能保证在任一阅读器里不会显示成乱码,且该emoji能显示在每个客户端表现形式也不统一。 我自己其实本地也做了许多的类似的美化,但深知别人不一定和我有一致的喜好,所以从来自用。如果不是功能上出现的bug,已有的路由的美化,还是只在本地做为好。 |
|
另外,这个 PR 改变了图片的输出,给图片添加了到原图的超链接,这也是不太有必要的。 |
|
说话间,又有人提相关的 issue 了。 #5927 |
|
一点小小的建议:这个形式也有道理,但是并非每个用户都需要,建议做成一个选项,或者新增一个路由。 |
|
我最近没什么时间,还是revert好,造成麻烦十分抱歉。 不过豆瓣广播原来的正文少了很多内容(主要是转发只显示原文)。待revert后我提个issue |
…)" This reverts commit 5a2e252.
|
其实不至于 Revert ,很多改进我是支持的,比如转发的内容是更加完整的👏,在转发上用 我觉得大家可以讨论一下是不是所有改动都不值得保留🤔。 |
|
@MisteryMonster 如果内容有变更,即使是加一个HTML节点这种变化都有可能让一些人的XPath崩掉。这个还是要权衡吧,看社区怎么反馈。我建议如果要保留那就以配置项的方式开启,默认还是和原来一样 我的阅读器是FreshRSS。图片大小和排版这种东西可能更依赖于阅读器,毕竟比如SimplePie这种库会过滤掉内联的style(?) |
|
建议在我们讨论和提出完善的解决方案前先revert吧,现在已经用不了了 (TдT) |
问题不在于改动值不值得保留,而是在于其直接大幅改动了默认输出,这导致了一连串的问题。 |
|
也是,大家不能用的确实比好看更加紧急,先 revert 。我们去 telegram 里面讨论吧? |
|
各位好,#6004 加入了 query string 开启可读性的选项,默认情况下不会有太大改动。 期待各位的讨论和反馈 |
目的 / What this PR does
该 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