@Jdlrobson, take a look at the attached screenshot. As you can see there is no room to the right of the image. If we want to make the close icon not overlap the image, then the image will have to be made smaller. I think the solution is to hide the close icon on tap (similar to the attribution text).

Point taken. I think the more serious part of this bug is the latter statement. I think if it lines up with the details button it won't look so strange.

I think it maybe best not to align the close icon with the details button because doings so will reduce the tappable area for the close icon and makes it hard for the user to close the image overlay on small screens.

@KHammerstein please advise on what we should do here..

@bmansurov @Jdlrobson
Button should align with details button, I think tap area is large enough. I added a mock to the description with them aligned and the button slightly smaller ~ 17 px square. Good point about how most interface disappears on tap. Apps also hide all buttons on tap. Lets do this too.

Also the arrows are very tiny and hard to see, I made them the same size as close in the mock and created a task to enlarge them here T108657

It is.

Maybe the following one, sprint 80. Is this one really, really simple? @Nirzar, you okay with us tackling this one?

I think it's low priority. is there space for 1-2 pointer? we can fix something that is broken?

Diid you fix this @bmansurov in your other patch?

@Jdlrobson, no that was another issue: T142118.

Change 306801 had a related patch set uploaded (by Jdlrobson):
Media viewer design tweaks

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

Change 306801 merged by jenkins-bot:
Media viewer design tweaks

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

You may test the change at https://en.m.wikipedia.beta.wmflabs.org/wiki/San_Francisco#/media/File%3ASF_From_Marin_Highlands3.jpg and https://en.m.wikipedia.beta.wmflabs.org/wiki/San_Francisco?mobileaction=beta#/media/File%3ASF_From_Marin_Highlands3.jpg. Good luck.