Bartosz Dziewoński has uploaded a new change for review.
https://gerrit.wikimedia.org/r/229671
Change subject: Revert "mediawiki.page.ready: When infusing, infuse by id, not
by element"
......................................................................
Revert "mediawiki.page.ready: When infusing, infuse by id, not by element"
Infusing by element is better, since it avoids the
document.getElementById() lookup and behaves better in case
of duplicate ids (which is invalid HTML, but happens).
This reverts commit 803d89173fc3a56871767663907785cb3eaf7aa5.
Depends on I11fd1c7f2434f904b9ddc94b0903f8278e06f3a1 in OOjs UI.
Bug: T105828
Change-Id: I8c1f5015229db7b8a391f324cfd96be4ac158041
---
M resources/src/mediawiki.page/mediawiki.page.ready.js
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/71/229671/1
diff --git a/resources/src/mediawiki.page/mediawiki.page.ready.js
b/resources/src/mediawiki.page/mediawiki.page.ready.js
index 70029a1..8ec4cf0 100644
--- a/resources/src/mediawiki.page/mediawiki.page.ready.js
+++ b/resources/src/mediawiki.page/mediawiki.page.ready.js
@@ -64,7 +64,7 @@
if ( $nodes.length ) {
mw.loader.using( 'mediawiki.widgets' ).done( function
() {
$nodes.each( function () {
- OO.ui.infuse( this.id );
+ OO.ui.infuse( this );
} );
} );
}
--
To view, visit https://gerrit.wikimedia.org/r/229671
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8c1f5015229db7b8a391f324cfd96be4ac158041
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
