wikimedia/mediawiki-extensions-Popups
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/Popups synced 2024-11-24 07:34:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

Move the article renderer in to the renderers property

Browse source 
To allow multiple renderers (for different types of content)
to be defined.

Change-Id: I50320646e26f36a0a2bc425c82ccc58912840f84
This commit is contained in:
Prateek Saxena 2015-08-11 19:15:24 +05:30 committed by Hoo man
parent faa464d19d
commit 1565a1d3c9
3 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
resources/ext.popups.renderer.article.js
View file

@ -558,6 +558,6 @@
}
};
mw.popups.render.article = article;
mw.popups.render.renderers.article = article;
} ) ( jQuery, mediaWiki );

8
resources/ext.popups.renderer.js
View file

@ -54,6 +54,12 @@
*/
mw.popups.render.currentRequest = undefined;
/**
* Object to store all renderers
* @property {Object} renderers
*/
mw.popups.render.renderers = {};
/**
* Close all other popups and render the new one from the cache
* or by finding and calling the correct renderer
@ -104,7 +110,7 @@
.done( function () {
// TODO: check for link type and call correct renderer
// There is only one popup type right now so it isn't necessary
var cachePopup = mw.popups.render.article.init( link );
var cachePopup = mw.popups.render.renderers.article.init( link );
mw.popups.render.openTimer = mw.popups.render.wait( mw.popups.render.POPUP_DELAY - mw.popups.render.API_DELAY );

4
tests/qunit/ext.popups.renderer.article.test.js
View file

@ -1,13 +1,13 @@
( function ( $, mw ) {
QUnit.module( 'ext.popups.renderer.article', QUnit.newMwEnvironment() );
QUnit.module( 'ext.popups.renderer.renderers.article', QUnit.newMwEnvironment() );
QUnit.test( 'render.article.getProcessedElements', function ( assert ) {
QUnit.expect( 13 );
function test ( extract, title, expected, msg ) {
var $div = $( '<div>' ).append(
mw.popups.render.article.getProcessedElements( extract, title )
mw.popups.render.renderers.article.getProcessedElements( extract, title )
);
assert.equal( $div.html(), expected, msg );
}