“您已退出登录”弹窗中的刷新按钮跳转到了主页

步骤:

  1. 登录
  2. 打开 Discourse 的另一个标签页
  3. 退出登录
  4. 切换到另一个标签页,查看上述消息
  5. 点击“刷新”

实际结果:

  1. 重定向到 https://meta.discourse.org

预期结果:

  1. 页面正常刷新 =)

浏览器:Firefox 76.0b8

(另外,似乎 GitHub 登录仅在第二次尝试时才有效?)

相关代码如下:

以及

logout 函数的引用是否可以直接替换为 window.location.reload

这里的行为不太理想,刷新应该……嗯,刷新,而不是“返回首页”。话虽如此,我怀疑实现是否正确,而按钮的标签可能标错了。如果我们不将用户带回家,刷新可能会导致 404 错误,特别是如果用户所在的页面是私信、安全分类、需要登录的网站等。

3 个赞

确实如此,不过就我而言,我是在阅读一些公开帖子。

另外,我猜这可能是一个问题:如果页面是私有的,那么在点击按钮之前,私有内容仍会隐约显示在对话框后面。因此,也许页面应该在点击后立即刷新?

这显然是有意为之。

客户端无法判断您查看的主题是否公开。我们可以在这里添加大量逻辑来实现这一功能,但在我看来,被登出是一种极其罕见的情况,在这种情况下,应将其视为“重置按钮”。

“重置按钮”应将您带回首页。

我倾向于支持更改这里的标签,但我不确定该叫什么——“刷新到首页”又太长了。

3 个赞

我想把“刷新”改成“首页”,这样我们就可以把这个话题标记为完成了,@jomaxro

2 个赞

浏览器“刷新”按钮表现良好,在必要时不会丢失我的位置。=)

PR 已提交:

4 个赞