Botão de atualizar no pop-up "Você foi desconectado." vai para a página inicial em vez disso

Passos:

  1. Faça login
  2. Abra outra aba do Discourse
  3. Faça logout
  4. Alterne para a outra aba para ver a mensagem acima
  5. Clique em “atualizar”

Resultado real:

  1. Redirecionado para https://meta.discourse.org

Resultado esperado:

  1. A página é atualizada =)

Navegador: Firefox 76.0b8

(Também, parece que o login do GitHub só funciona na segunda tentativa?)

O código relevante parece ser:

e

A referência à função logout poderia ser simplesmente substituída por window.location.reload?

O comportamento aqui não é ideal; a atualização deveria, bem, atualizar, não “ir para a página inicial”. Dito isso, me pergunto se a implementação está correta e se o botão está com o rótulo errado. Se não enviarmos as pessoas para a página inicial, a atualização pode resultar em erros 404, caso a página em que estavam fosse uma mensagem privada, uma categoria segura, um site com login obrigatório, etc.

3 curtidas

Isso é verdade, embora no meu caso eu estivesse lendo vários tópicos públicos.

Também acho que pode ser um problema que, se a página fosse privada, o conteúdo privado ainda ficasse meio visível atrás do diálogo antes de o botão ser clicado — então talvez a página devesse ser atualizada imediatamente?

Isso certamente é intencional.

O cliente não tem como saber se o tópico que você estava procurando é público ou não. Poderíamos implementar muita lógica aqui para permitir isso, mas, na minha visão, ser desconectado é uma condição extremamente rara, e, nesse caso, deve ser considerado um “botão de reinicialização”.

Um “botão de reinicialização” deve levá-lo de volta à página inicial.

Acho que apoio a alteração do rótulo aqui, mas não tenho certeza do que chamaríamos. “Atualizar para a Página Inicial” é apenas muito longo.

3 curtidas

Acho que podemos mudar “Atualizar” para “Início” e então encerrar este tópico como concluído @jomaxro?

2 curtidas

O botão ‘atualizar’ do navegador funciona bem e não perde meu lugar se não for necessário. =)

PR está no ar:

4 curtidas