Katie Horn has submitted this change and it was merged.
Change subject: Prevent Google indexing of the CentralNotice div
......................................................................
Prevent Google indexing of the CentralNotice div
We're doing this here because it takes too long to roll out a skin change.
Change-Id: Id6bf279e590409b3464c363ee16442f4eb0dc186
---
M modules/ext.centralNotice.bannerController/bannerController.js
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Katie Horn: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ext.centralNotice.bannerController/bannerController.js
b/modules/ext.centralNotice.bannerController/bannerController.js
index 1ed85af..26891cd 100644
--- a/modules/ext.centralNotice.bannerController/bannerController.js
+++ b/modules/ext.centralNotice.bannerController/bannerController.js
@@ -443,7 +443,7 @@
mw.centralNotice.bannerData.bannerName =
bannerJson.bannerName;
$( 'div#centralNotice' )
.attr( 'class', mw.html.escape( 'cn-' +
mw.centralNotice.data.category ) )
- .prepend( bannerJson.bannerHtml );
+ .prepend( '<!--googleoff: all-->' +
bannerJson.bannerHtml + '<!--googleon: all-->' );
// Create landing page links if required
if ( bannerJson.autolink ) {
--
To view, visit https://gerrit.wikimedia.org/r/177611
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id6bf279e590409b3464c363ee16442f4eb0dc186
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CentralNotice
Gerrit-Branch: master
Gerrit-Owner: Awight <[email protected]>
Gerrit-Reviewer: AndyRussG <[email protected]>
Gerrit-Reviewer: Awight <[email protected]>
Gerrit-Reviewer: Ejegg <[email protected]>
Gerrit-Reviewer: Katie Horn <[email protected]>
Gerrit-Reviewer: Mwalker <[email protected]>
Gerrit-Reviewer: Ssmith <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
