Jdlrobson has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/404603 )

Change subject: Summaries should not contain coordinates
......................................................................

Summaries should not contain coordinates

Bug: T185050
Change-Id: Idcb882eeabad8c63c18cf0e94a25ca799899445d
---
M lib/summary.js
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps 
refs/changes/03/404603/1

diff --git a/lib/summary.js b/lib/summary.js
index 62c57ce..6b6480c 100644
--- a/lib/summary.js
+++ b/lib/summary.js
@@ -91,6 +91,8 @@
     if (isMainPage) {
         return EMPTY_EXTRACTS;
     } else {
+        // The summary should not present any html from removed elements e.g. 
geo tag
+        transforms.stripUnneededMarkup(doc)
         const leadSectionDoc = parsoidSections.justLeadSection(doc);
         const intro = transforms.extractLeadIntroduction(leadSectionDoc);
         if (intro.length) {

-- 
To view, visit https://gerrit.wikimedia.org/r/404603
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idcb882eeabad8c63c18cf0e94a25ca799899445d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to