Passaggi:
Risultato effettivo:
Risultato atteso:
Browser: Firefox 76.0b8
(Inoltre, sembra che il login tramite GitHub funzioni solo al secondo tentativo?)
Il codice rilevante sembra essere:
e
Il riferimento alla funzione logout potrebbe essere semplicemente sostituito con window.location.reload?
Il comportamento qui non è ottimale: il tasto di aggiornamento dovrebbe, beh, aggiornare, non “tornare alla home”. Detto questo, mi chiedo se l’implementazione sia corretta e se sia il pulsante ad essere etichettato male. Se non riportiamo gli utenti alla home, l’aggiornamento potrebbe generare errori 404, nel caso in cui la pagina su cui si trovavano fosse un messaggio privato, una categoria sicura, un sito con login_required, ecc.
È vero, anche se nel mio caso stavo leggendo una serie di discussioni pubbliche.
Inoltre, immagino che possa essere un problema il fatto che, se la pagina è privata, il contenuto privato sia comunque parzialmente visibile dietro la finestra di dialogo prima che venga cliccato il pulsante; forse la pagina dovrebbe essere ricaricata immediatamente?
Questo è sicuramente intenzionale.
Il client non ha modo di sapere se l’argomento che stavi cercando è pubblico o meno; potremmo implementare molta logica qui per gestire questo caso, ma, a mio avviso, essere disconnessi è una condizione estremamente rara e, in tali circostanze, dovrebbe essere considerato come un “pulsante di reset”.
Un “pulsante di reset” dovrebbe riportarti alla home.
Credo di essere favorevole a cambiare l’etichetta qui, ma non sono sicuro di cosa potremmo chiamarla: “Aggiorna alla pagina iniziale” è semplicemente troppo lungo.
Immagino che cambiando “Aggiorna” in “Home” possiamo chiudere questo argomento come completato @jomaxro?
Il pulsante “aggiorna” del browser funziona bene e non perde la mia posizione se non è necessario. =)
La PR è stata aggiornata:
