Page MenuHomePhabricator

Newcomer tasks: arrows are different distances from card on Mobile
Open, Needs TriagePublic

Description

@PPham noticed that the "forward" and "back" arrows next to the suggested edit cards are slightly different distances from the card. The "forward" arrow is closer than the back arrow:

Both arrows should only be 16px from the card:

Event Timeline

It's the same in the desktop version now...

It's the same in the desktop version now...

Unfortunately it is still not quite evenly spaced:


I think this is being caused by a negative margin-left on the class .oo-ui-buttonElement-frameless.oo-ui-iconElement:first-child

Also, the arrows should always be contained within the module even when it is extremely narrow as shown in these screenshots. I have added this restyling for narrow width SE module to the same responsive article cards ticket T238598

Change 565455 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/extensions/GrowthExperiments@master] Fix suggested edits arrow positioning

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

Change 565455 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Fix suggested edits arrow positioning

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

Checked in betalabs - the arrows are positioned equally

Also, the arrows should always be contained within the module even when it is extremely narrow as shown in these screenshots. I have added this restyling for narrow width SE module to the same responsive article cards ticket T238598

It's also done.

Kosta pointed out on the patch that there's still a tiny difference (the left arrow is 2px farther).

Kosta pointed out on the patch that there's still a tiny difference (the left arrow is 2px farther).

Yes there is still a discrepancy - it looks like this may be due to uneven positioning being added on the span element of the arrows?

@RHo -- do we still have this issue? If not, please resolve.

@RHo -- do we still have this issue? If not, please resolve.

This is still an issue on Mobile:

Also, I think there is a regression elsewhere too because now on Mobile the arrows are further away from the card than expected:

(expected Zeplin mock)

RHo renamed this task from Newcomer tasks: arrows are different distances from card to Newcomer tasks: arrows are different distances from card on Mobile.Oct 8 2020, 5:29 PM
RHo updated the task description. (Show Details)