Page MenuHomePhabricator

Display text so that it doesn't hide the image
Open, Needs TriagePublic

Description

Currently, some information is displayed as a caption covering the lower part of the image.
It is sometimes problematic because it hides important elements that need to be depicted. Here is an example where almost half the image is hidden.

Screenshot_20191007-230935_Fennec F-Droid.jpg (1,200×1,920 px, 552 KB)

It would be great if this information was displayed elsewhere, for example with the rest of the other information just below :)

Thanks in advance!

Details

Event Timeline

For more clarity, here is what the image looks like in full.

Screenshot_20191007-231656_Fennec F-Droid.jpg (1,200×1,920 px, 696 KB)

Change 575216 had a related patch set uploaded (by Gabrielchihonglee; owner: Gabrielchihonglee):
[labs/tools/Isa@master] Fix image overlay text covering the image

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

It still looks like the "before" version on my mobile.

It still looks like the "before" version on my mobile.

You mean even after applying this patch: https://gerrit.wikimedia.org/r/#/c/labs/tools/Isa/+/575216/ ?

@Gabrielchihonglee the screenshot below is a situation where the File attribute of the image is blocked partially following your recent change.

Screen Shot 2020-03-05 at 12.11.37.png (284×612 px, 180 KB)

You may find my comment on the PS helpful :)

Yeah, I saw it but it doesn't sound like a great / proper method to solve the issue.

Change 575216 abandoned by Eugene233:

[labs/tools/Isa@master] Fix image overlay text covering the image

Reason:

PS not worked on for a while.

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

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly point to the codebase URL and provide clear steps to help a contributor get set up for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

Pppery added a subscriber: Hype4shreshth.
Shashwatojha369 subscribed.

Hello, I'd like to work on this task.

I investigated the issue and found that the license/author information is displayed through .image-overlay inside .img-holder and is positioned using absolute positioning, which appears to cause the information to cover part of the image.

I'm reviewing the previous abandoned patch and testing possible solutions to display this information without obscuring the image, including checking the behavior on mobile devices.

Thanks!

Hello! I have submitted a patch for this task.

Gerrit change: https://gerrit.wikimedia.org/r/c/labs/tools/Isa/+/1308348

This patch moves the image attribution (author, credit, and license) below the image so it no longer obscures important parts of the image while keeping the attribution visible.

Thank you for reviewing!

@Shashwatojha369 thank you for your contribution. However, ISA was migrated to gitlab a while ago.