Merge "Revert "mmv.js: Store comingFromHashChange as a class property""

This commit is contained in:
jenkins-bot 2024-11-13 23:17:05 +00:00 committed by Gerrit Code Review
commit a0403ff80e

View file

@ -44,6 +44,7 @@ const ThumbnailWidthCalculator = require( './mmv.ThumbnailWidthCalculator.js' );
const { extensions, useThumbnailGuessing } = require( './config.json' );
const router = require( 'mediawiki.router' );
let comingFromHashChange = false;
/**
* Analyses the page, looks for image content and sets up the hooks
@ -95,7 +96,7 @@ class MultimediaViewer {
*/
this.router = router;
this.setupRouter();
this.comingFromHashChange = false;
comingFromHashChange = false;
/**
* UI object used to display the pictures in the page.
@ -685,8 +686,8 @@ class MultimediaViewer {
this.viewLogger.recordViewDuration();
this.viewLogger.unattach();
if ( this.comingFromHashChange ) {
this.comingFromHashChange = false;
if ( comingFromHashChange ) {
comingFromHashChange = false;
} else {
this.router.back();
}
@ -706,7 +707,7 @@ class MultimediaViewer {
// handle empty hashes, and anchor links (page sections, possibly including /)
this.router.addRoute( /.*$/, () => {
if ( this.isOpen ) {
this.comingFromHashChange = true;
comingFromHashChange = true;
document.title = this.createDocumentTitle( null );
if ( this.ui ) {
// FIXME triggers mmv-close event, which calls viewer.close()