Visit https://en.m.wikipedia.org/wiki/Leonard_Cohen?oldid=747517267
The box in the top says that the last modifier is Linealchamp.
{F8393597}
The last modified bar at the end disagrees.
{F8393602}
It shows the current editor too.
We should hide the bar in the footer which gives information that is not relevant (not appropriate) here.
= Acceptance criteria
[x] If a page is an old revision and I view it there is no last modified bar
[x] If a page is a current revision and I view it there is a last modified bar.
[X] MinervaNeue should not use MobilePage class - use methods in core.
= Developer notes
[] MinervaNeue uses https://github.com/wikimedia/mediawiki-extensions-MobileFrontend/blob/master/includes/models/MobilePage.php. It might be good to disentangle the dependency here as part of this change as relying on this means Minerva depends on MobileFrontend.
[] Rely on internal Revision methods rather than query strings parameters.
[] Take the time to review the codebase and refactor so we consistently check if the page is a revision
= Testing notes
[] Go to any article on BetaCluster that has more than one edit, example: https://en.m.wikipedia.beta.wmflabs.org/wiki/Planet
[] The footer should contain "Last edited {TIME} ago by {AUTHOR}
[] click "Last edited" link to see the article history page
[] on history page click the previous edit (not the latest one)
[] you should land on page saying "This is an old revision of this page ....."
[] when you scroll down to the footer "Last edited {TIME} ago by {AUTHOR}" should not be visible because you're viewing the historical page