Page MenuHomePhabricator

Large gap left in footer on pages with no related articles
Closed, ResolvedPublic1 Estimated Story Points


When visiting (all browserS)
An empty container is left on the page despite there being no related articles. This leads to a large gap in the footer

Expected: The container should be removed in this instance
When vistiting
Expected: the container should be present

<div class="read-more-container"></div>


Screen Shot 2016-10-03 at 11.12.29 AM.png (276×1 px, 30 KB)


Screen Shot 2016-10-03 at 11.13.21 AM.png (269×1 px, 30 KB)

Requirement for task
Install and setup MobileFrontend on MediaWiki.

Reproduce this bug
Open an article that doesn't exist on English Wikipedia in mobile view and you will notice a container gap at the top of the page. Example Me article or any other page with no related article. You will see a page with a gap at the left footer on the page (as images show above).

  • Reproduce locally**

You can make the related articles show on a page by adding the following wikitext

{{#related:Page title}}

Event Timeline

I don't see it as a blocker - we can pull it into next sprint. @Nirzar - let me know if you're okay with this as well.

yeah.. this is fine. the gap is actually supposed to be there. it's just a visual detail to play with whitespace and branding.

@Nirzar are you sure you are not misunderstanding the bug?

If we don't remove this, the gap is inconsistent between special pages and pages without related articles. We're not talking about removing the gap altogether.

@Jdlrobson oh yes the inconsistency is an issue. ideally there shouldn't be any padding based on an optional element (related pages here)

i take my "invalid" back. this is low priority bug. the container should not be present as written in the description. sorry about the confusion.

not a blocker for footer release. triaged but future?

Nirzar lowered the priority of this task from High to Low.Oct 5 2016, 9:36 PM
Nirzar added a project: Design.
MBinder_WMF set the point value for this task to 1.Oct 11 2016, 4:33 PM
Jdlrobson updated the task description. (Show Details)
divadsn added a subscriber: divadsn.

I am working to fix this, patch following soon.

I have problems reproducing the bug, I can't find anything in the code where this div container could appear. Am I cloning the wrong repository?

Hi @divadsn you'll need to be looking in Extension:RelatedArticles
Specifically for where read-more-container is added.
Hope that helps! Feel free to ask any more question.

@Jdlrobson, thank you, I was on the wrong way :)

Change 326252 had a related patch set uploaded (by Divadsn):
Large gap left in footer on pages with no related articles

Change 326252 merged by jenkins-bot:
Large gap left in footer on pages with no related articles

Approved and signed off. I tested on Vector/Minerva for non-existent pages and pages which had related articles.