Étapes :
Résultat réel :
Résultat attendu :
Navigateur : Firefox 76.0b8
(De plus, il semble que la connexion via GitHub ne fonctionne qu’au deuxième essai ?)
Le code pertinent semble être :
et
La référence à la fonction logout pourrait-elle simplement être remplacée par window.location.reload ?
Le comportement ici n’est pas idéal : l’actualisation devrait, eh bien, actualiser, et non « retourner à l’accueil ». Cela dit, je me demande si l’implémentation est correcte et si le bouton est mal libellé. Si nous ne renvoyons pas les utilisateurs à l’accueil, l’actualisation pourrait entraîner des erreurs 404, notamment si la page sur laquelle ils se trouvaient était une messagerie privée, une catégorie sécurisée, un site nécessitant une connexion, etc.
C’est vrai, bien que dans mon cas, je lisais un tas de discussions publiques.
Je pense aussi que cela pourrait poser problème si la page était privée, car le contenu privé reste en quelque sorte visible derrière la fenêtre de dialogue avant que le bouton ne soit cliqué. Peut-être que la page devrait donc se rafraîchir immédiatement ?
C’est certainement intentionnel.
Le client n’a aucun moyen de savoir si le sujet que vous cherchiez est public ou non. Nous pourrions intégrer beaucoup de logique ici pour permettre cela, mais, selon moi, être déconnecté est une condition extrêmement rare et, dans un tel cas, cela devrait être considéré comme un « bouton de réinitialisation ».
Un « bouton de réinitialisation » doit vous ramener à l’accueil.
Je penche pour la modification de l’étiquette ici, mais je ne suis pas sûr de ce que nous pourrions l’appeler. « Actualiser vers la page d’accueil » est tout simplement trop long.
Je suppose qu’il faut remplacer « Rafraîchir » par « Accueil », puis nous pouvons clôturer ce sujet comme étant terminé @jomaxro ?
Le bouton « actualiser » du navigateur fonctionne bien et ne me fait pas perdre ma place si ce n’est pas nécessaire. =)
La PR est en ligne :
