sam
(Sam Saffron)
107
我其实反对在这里设置站点选项,不过如果你愿意的话,可以发布一个主题组件。
4 个赞
riking
(Kane York)
109
此外,在清理工作(将 bookmarkWithReminder 的用法改为 bookmark)完成之前,任何旨在恢复旧行为的自定义修改都将不稳定。
1 个赞
sam
(Sam Saffron)
110
我认为也值得提一下,旧的行为是:我们有一个几乎没人点击的按钮,执行了一个大家其实并不在意的操作。我们有大量统计数据可以佐证这一点。
我非常反对“恢复旧行为”,因为人们……有他们的理由……
如果你真的讨厌书签,为什么不直接禁用书签呢?
10 个赞
是的,没错,如果你真的想彻底移除书签功能,只需从帖子的命令列表中将其删除即可。这个功能之前本来就没多大用处……它唯一能派上用场的地方就是实现跨浏览器、跨操作系统的“书签”功能,但这实在是一个非常狭窄的使用场景。
7 个赞
pnoeric
(Eric)
112
好的,那么按照“主题组件”的路径来实现是正确的做法吗?我不会试图说服大家这样做……但我希望可以请教一下,如何在我的环境中实现这一功能。
收到。
1 个赞
martin
(Martin Brennan)
121
今天收获颇丰。以下 PR 已合并,除最后一个外,其余均已上线 Meta:
此外,FIX: Do not save bookmark if close (X) on modal is clicked by martin-brennan · Pull Request #9541 · discourse/discourse · GitHub 预计很快也将合并。我还提交了一个 PR,旨在将 /bookmarks-with-reminders 的 URL 改回 /bookmarks,希望明天能合并该 PR:https://github.com/discourse/discourse/pull/9566。
“主题用户书签列逻辑不正确”的修复预计将在几小时内上线 Meta。本周晚些时候,我们将为所有这些修复推出另一个 Beta 版本。
18 个赞
martin
(Martin Brennan)
122
另一个更新——这两个 PR 现在都已合并,没有遗留的 bug。我现在正在处理最后一个 PR,将变量名等(如 bookmarkWithReminder)改回 bookmark。
12 个赞
我已在“主题列表预览”插件中实现了从主题列表直接单击、无需对话框即可进行书签操作的功能(该插件仍允许您在主题中使用丰富的书签功能)。
由于我需要紧跟核心更新的变化,功能可能在几天内不够稳定,但昨天它已经可以正常工作了!
5 个赞
martin
(Martin Brennan)
128
最终的 PR 已合并。我们预计在未来一两天内发布新的测试版。新书签功能目前没有任何遗留问题:
。
14 个赞
martin
(Martin Brennan)
130
4 个赞
martin
(Martin Brennan)
147
我刚合并了另一个更改,该更改将在几小时内上线 Meta。现在,在书签弹窗中有一个隐藏选项,可在发送提醒后删除书签。我们已在本地存储中记录您选择的选项,因此此行为对您创建的每个书签都将保持一致。
9 个赞
Dannii
(Dannii)
166
人们会使用浏览器书签的想法让我觉得好笑,所以我很高兴 Discourse 仍然保留了书签功能。我经常在这里为帖子添加书签,但绝不会想到在浏览器中添加书签。我想我有点奇怪。
3 个赞
martin
(Martin Brennan)
174
我又提交了一个 PR 来修复这个问题:https://meta.discourse.org/t/domain-com-bookmarks-is-showing-domain-com-u-user-activity-bookmarks-with-reminders/149252,同时还在书签列表中添加了被书签帖子的用户头像。该 PR 即将合并:
@Ellibereth,我记得你之前提到过希望看到头像;@exetico,感谢你对 URL 变更的反馈。
11 个赞
如果能更多地利用“此书签用途”字段的信息就好了。目前,查看该信息的唯一方式是通过个人资料中的完整书签列表或打开书签本身。我认为,如果该信息能在下拉菜单的书签列表中使用,或者包含在提醒通知中,将会更加实用。
4 个赞
提醒通知不包含任何额外信息,那么你觉得这该怎么实现?难道要为了这个专门增加一个全新的功能区域吗?
3 个赞
其实我在想,如果该字段设置了文本,是否应该用它来替换通知中的主题标题。只是随口一提。
7 个赞
martin
(Martin Brennan)
178
我部分同意你的看法。这可能不太明显,但当你将鼠标悬停在提醒通知上时,可以看到“此书签的用途”信息。我们在通知时已经包含了该信息,但不想仅为这一功能创建一种不同的通知格式。
9 个赞