Hello,
in Firefox, the previous page title that appears with a right clic on the “previous page” arrow is overwritten with the current page title.
For instance, if the first page displayed is:
https://meta.discourse.org/
see what the right click on arrow looks like (it is correct):
then going on any topic, such as:
the title of the previous page has been overwritten with the current page title:
Nevertheless, the link is still valid.
any idea about this bug?
It does indeed look like a bug possibly in the way / sequence that we replace history or a firefox quirk.
I am able to reproduce.
Since it is fairly minor I am going to put a pr-welcome in case the community would like to pitch in and propose a fix.
alright, thanks for your answer
He descubierto que este error también se manifiesta como entradas en el historial de navegación que están algo corruptas, lo cual considero un problema más grave. Más específicamente, se asigna un nuevo título a una URL antigua cuando la navegación es manejada por JS. Esto ocurre casi siempre en Firefox, pero es bastante raro en Chrome (he encontrado una instancia hasta ahora: al pasar de /top a un tema).
@sam tiene razón en que se debe al orden de las operaciones en las que se modifican la URL y el título. Ojalá hubiera alguna guía de mejores prácticas publicada en algún lugar que dijera cuál es el orden correcto: siempre actualizar el título después de usar la API history.
He intentado depurar un poco (y quiero decir de antemano que soy bastante reacio a construir cualquier proyecto medianamente grande adyacente a npm). Inicialmente usé este código para hacer un pequeño reconocimiento:
Object.defineProperty(document, "title", {
get() {
return document.head.querySelector('title').innerText;
},
set(t) {
console.debug('title', t, new Error("title!").stack)
debugger;
document.head.querySelector('title').innerText = t;
},
enumerable: true,
configurable: true,
});
let ___push___ = window.history.pushState;
window.history.pushState = function(...args) {
console.debug('push', args[0], new Error().stack);
debugger;
___push___.apply(this, args);
};
let ___replace___ = window.history.replaceState;
window.history.replaceState = function(...args) {
console.debug('replace', args[0], new Error().stack);
debugger;
___replace___.apply(this, args);
};
Efectivamente, demostró que casi siempre hay una modificación del título justo antes de la operación de la API history. Desafortunadamente, no fue más útil que eso: relacionar lo que sucede en tiempo de ejecución con el código fuente fue bastante difícil – las operaciones de interés están bastante dispersas, la pila de llamadas era engañosa porque la ejecución ya había cedido al bucle de eventos de Ember, y en general la experiencia de depuración en Firefox fue bastante infernal. Al cambiar a Chrome, tuve un poco más de éxito. En resumen, hasta donde sé, Ember decide por sí mismo cuándo actualizar la URL. Por otro lado, la modificación del título puede iniciarse desde dos lugares: [1] y [2]
Solo el último se ejecuta antes de la llamada a la API history. El ‘ancestro de código común’ de ellos se encuentra aquí:
// Ejecutar todos los hooks necesarios de entrada/configuración/salida
this.setupContexts(newState, transition); // <- cambio de título iniciado aquí, al activar el evento `model.title:change`
// Comprobar si se produjo una redirección en la entrada/configuración
if (transition.isAborted) {
// TODO: ¿forma más limpia? ¿distinguir entre routeInfos de destino?
this.state!.routeInfos = this.currentRouteInfos!;
return Promise.reject(logAbort(transition));
}
this._updateURL(transition, newState); // <- cambio de URL iniciado aquí
Curiosamente, “router.js es la microbiblioteca de enrutamiento utilizada por Ember.js”.
Ahora me gustaría ceder el manejo del problema a alguien que, a diferencia de mí, esté remotamente familiarizado con la base de código de discourse.
Editar: olvidé mencionar: document.title es modificado por [1] una o más veces después de (ambos) ser modificado [2] y la URL ser modificada por la API history.
(Como usuario nuevo, no puedo poner más de 2 enlaces por publicación 
)