wikimedia/mediawiki-extensions-Math
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/Math synced 2024-12-18 10:30:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix for Preview has wrong location in MathML mode

Browse source 
The previewin MathML mode was always at the left side of the website.
Added values for `node.offsetWidth` and `node.offsetHeight`.
Values are taken from `node.getBoundingClientRect()`.
Cleaned up if-clause(node.dataset.title) as it is not needed anymore.
In-lined IsValidId funtion as there was only one remaining reference.

Bug: T381311
Change-Id: Ied17aaaaf9156e2712ed6c7d6d6f08ecccc12286
This commit is contained in:
FrederikHennecke1 2024-12-02 20:38:33 +01:00
parent 500b9bcefc
commit b66e28b237
1 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
modules/ext.math.popup.js
View file

@ -3,9 +3,6 @@
const previewType = 'math';
const selector = '.mwe-math-element[href*="qid"], .mwe-math-element[data-qid] img';
const api = new mw.Rest();
const isValidId = function ( qid ) {
return qid.match( /Q\d+/g ) === null;
};
const fetch = function ( qid ) {
return api.get( '/math/v0/popup/html/' + qid, {}, {
Accept: 'application/json; charset=utf-8',
@ -29,7 +26,7 @@
if ( parent.dataset.qid ) {
qidstr = parent.dataset.qid;
}
if ( !qidstr || isValidId( qidstr ) ) {
if ( !qidstr || ( qidstr.match( /Q\d+/g ) === null ) ) {
return deferred.reject();
}
qidstr = qidstr.slice( 1 );
@ -48,13 +45,15 @@
} );
return deferred.promise();
};
// popups require title attributes
// popups require offsetHeight and offsetWidth attributes
[].forEach.call(
document.querySelectorAll( selector ),
( node ) => {
const qidstr = getQidStr( node );
if ( qidstr && isValidId( qidstr ) ) {
node.dataset.title = 'math-unique-identifier';
if ( typeof node.offsetWidth === 'undefined' ) {
node.offsetWidth = node.getBoundingClientRect().width || 1;
}
if ( typeof node.offsetHeight === 'undefined' ) {
node.offsetHeight = node.getBoundingClientRect().height || 1;
}
}
);