This is ready for you to take a look at @Jdrewniak. I think we should go for items 1, 2, and 3 in the bug section, and items 1, 2, and 3 in the refinements section.

Reviewed this and it looks good! Currently, not considering it a blocker for deployment

Change 528201 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/core@master] move rcfilters scroll logic to better support mobile mode

https://gerrit.wikimedia.org/r/528201

So far:
Bugs:
1 Page should not scroll horizontally
3 When tapping Hide the entire filter box should be hidden.
Are taken care of in this patch:
https://gerrit.wikimedia.org/r/527132

2 When tapping on gray Active filters area (to open filters menu) page should scroll down to reveal filters menu
is taken care of in this patch
https://gerrit.wikimedia.org/r/528201

4 Save a filter behaviour - this needs some more investigation so I'll look into it after some the lower hanging fruit in the refinements.

Opportunities for refinement
1, 3, 5 can probably go in one patch since the changes can be contained in our changeslist override styles.
2 is a little more involved but doable.
4 and 6 can be quite fiddly so we might not get to these.

Change 527132 merged by jenkins-bot:
[mediawiki/core@master] rcfilters mobile cleanup

https://gerrit.wikimedia.org/r/527132

Change 528220 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Add explanatory comment to unexpected line of code

https://gerrit.wikimedia.org/r/528220

Change 528225 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Log entries are wrapped with parenthesis - not hardcoded into HTML

https://gerrit.wikimedia.org/r/528225

During review we noticed that show/hide causes the box to shrink in size but that appears to be intentional (https://phabricator.wikimedia.org/T177206#4271707) so I've added in explanatory comment

Fixed by: https://gerrit.wikimedia.org/r/528220

Also noticed these stray brackets...

.. and log entries being broken
Text here should read " Toby talk contribs uploaded File:Another Screenshot.png"

this is fixed by: https://gerrit.wikimedia.org/r/528225
After:

Change 528201 merged by jenkins-bot:
[mediawiki/core@master] move rcfilters scroll logic to better support mobile mode

https://gerrit.wikimedia.org/r/528201

Change 528220 merged by jenkins-bot:
[mediawiki/core@master] Add explanatory comment to unexpected line of code

https://gerrit.wikimedia.org/r/528220

@Jdrewniak @Jdlrobson noticing on beta, for both iOS (Safari) and Android (Chrome) the filter menu is squishing up if you scroll after it opens initially. This was an issue early on, though I'm pretty sure it was fixed at some point.

iOS	Android

Change 528225 merged by jenkins-bot:
[mediawiki/core@master] Improve log entries display in change lists

https://gerrit.wikimedia.org/r/528225

The following should be now fixed:

1. Page should not scroll horizontally (note: this may be fixed by reducing the base font-size to 14px, which is item #1 in the "Opportunities for refinement" section below)

When tapping on gray Active filters area (to open filters menu) page should scroll down to reveal filters menu

When tapping Hide the entire filter box should be hidden. Currently the Filter, Namespaces, and Tags buttons are still shown, but are disabled.

Display issues with log entries (https://phabricator.wikimedia.org/T229360#5394174)

Increase the font-size and padding-top for the date headings in the list view

List entries should be left-aligned

Remaining issues (not fixed yet, possibly need bugs):

Save filter and trashcan buttons should be repositioned so that 1) the Active filter area takes up less vertical space, 2) the "No active filters state" (after
pressing the trashcan) looks less weird

Filter menu should match the width of the Filter button

Filter menu resizes on scrolling https://phabricator.wikimedia.org/T229360#5394461

Highlighting works inconsistently between desktop and mobile

Unnecessary commas display between all the edit tags

@alexhollender can you confirm (and that this list covers everything we've found so far?) and I'll cut bugs.

Change 528438 had a related patch set uploaded (by Jdrewniak; owner: Jdrewniak):
[mediawiki/skins/MinervaNeue@master] Recent changes design cleanup - font-size changes

https://gerrit.wikimedia.org/r/528438

@Jdlrobson I didn't realize the work here should be split off into separate tasks, so I attached the last patch to this task instead (it's small though). The patch I submitted above https://gerrit.wikimedia.org/r/528438 addresses refinements 1, 3, 5.

I think the large portions of work that can be split off are are:

• bug #4 - "Set as default" behaviour
• refinement #4 - "remove highlight functionality from rcfilters mobile view"
• "Unnecessary commas between edit tags" - this involves modifying the core html output (like Jon & Volker did with the history page) I think this is a nice to have. Maybe we can get rid of the semicolon separator in the article title while we're at it :P

There are a few that I don't think are worth fixing:

• the filter menu width matching the input width (improvements #6) - this might be a bug with the OOUI clippable class.
• "Filter menu resizes on scrolling" - this only happens when you scroll down and the browser chrome hides. The menu then appears slightly smaller then it should, but there is still plenty of space to use that menu.

Change 528438 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Recent changes design cleanup - font-size changes

https://gerrit.wikimedia.org/r/528438

@alexhollender
bugs: 1, 2, 3 & "Display issues with log entries T229360#5394174"
enhancements: 1, 3, 5
are live on beta.

Marking epic as this task covers several parts of the feature and we'll probably want to consider each bullet point separately.

Previously the Namespaces and Tags were sitting side-by-side, but currently they are each taking up their own line

Is this a problem? My understanding was this was needed as part of the work in T225499 to make sure the filters didn't spill out the page.

Per discussion: moving to kanbanana (needs analysis). To analyst: please decide whether to keep this as a single task or split it into four. Once the task has been analyzed, it will need an estimate as per usual. We think this is still in scope for AMC.

Diff hyperlinks from Minerva links to the desktop site for some reason. Should I create a separate task for it?

In T229360#5629198, @Masumrezarock100 wrote:

Diff hyperlinks from Minerva links to the desktop site for some reason. Should I create a separate task for it?

This is well documented in T156847 and several related tasks. These links will resolve to mobile if you are on the mobile site so only impact users of the mobile site on a desktop browser.

In T229360#5632130, @Jdlrobson wrote:

In T229360#5629198, @Masumrezarock100 wrote:

Diff hyperlinks from Minerva links to the desktop site for some reason. Should I create a separate task for it?

This is well documented in T156847 and several related tasks. These links will resolve to mobile if you are on the mobile site so only impact users of the mobile site on a desktop browser.

Well I sometimes have to use the desktop site in my mobile view. And after clicking Desktop link at the site footer, the desktop links don't redirect me to the mobile site anymore unless clear my cookies/use incognito mode.

Yes if you are switching between experiences that will happen. T156847 has seen little progress/consensus in several years though but is the place to talk about this bug.

Moving this off the board as it's not an explicit goal for this quarter