步骤:
- 登录
- 打开 Discourse 的另一个标签页
- 退出登录
- 切换到另一个标签页,查看上述消息
- 点击“刷新”
实际结果:
预期结果:
- 页面正常刷新 =)
浏览器:Firefox 76.0b8
(另外,似乎 GitHub 登录仅在第二次尝试时才有效?)
步骤:
实际结果:
预期结果:
浏览器:Firefox 76.0b8
(另外,似乎 GitHub 登录仅在第二次尝试时才有效?)
相关代码如下:
以及
对 logout 函数的引用是否可以直接替换为 window.location.reload?
这里的行为不太理想,刷新应该……嗯,刷新,而不是“返回首页”。话虽如此,我怀疑实现是否正确,而按钮的标签可能标错了。如果我们不将用户带回家,刷新可能会导致 404 错误,特别是如果用户所在的页面是私信、安全分类、需要登录的网站等。
确实如此,不过就我而言,我是在阅读一些公开帖子。
另外,我猜这可能是一个问题:如果页面是私有的,那么在点击按钮之前,私有内容仍会隐约显示在对话框后面。因此,也许页面应该在点击后立即刷新?
这显然是有意为之。
客户端无法判断您查看的主题是否公开。我们可以在这里添加大量逻辑来实现这一功能,但在我看来,被登出是一种极其罕见的情况,在这种情况下,应将其视为“重置按钮”。
“重置按钮”应将您带回首页。
我倾向于支持更改这里的标签,但我不确定该叫什么——“刷新到首页”又太长了。
我想把“刷新”改成“首页”,这样我们就可以把这个话题标记为完成了,@jomaxro?
浏览器“刷新”按钮表现良好,在必要时不会丢失我的位置。=)
PR 已提交: