Page MenuHomePhabricator

Discover section UI
Closed, ResolvedPublic

Assigned To
Authored By
hueitan
Mar 14 2022, 1:29 PM
Referenced Files
F35060516: image.png
Apr 20 2022, 7:35 PM
F35052686: image.png
Apr 15 2022, 12:33 PM
F35052806: image.png
Apr 15 2022, 12:33 PM
F35005287: image.png
Mar 14 2022, 1:29 PM

Description

image.png (780×360 px, 116 KB)

Event Timeline

hueitan moved this task from Backlog to Dev on the Inuka-Team (Kanban) board.

Change 770594 had a related patch set uploaded (by Huei Tan; author: Huei Tan):

[mediawiki/extensions/Wikistories@master] Discover Section UI

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

Change 770594 merged by jenkins-bot:

[mediawiki/extensions/Wikistories@master] Discover Section UI

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

Testing:
Due to the position of the "Create Wikistory" text, it makes it seem the padding between a story and the CTA to create a story is insufficient. Perhaps the text should be made to be in the design to correct that impression. See the below screenshots for the difference.

image.png (922×608 px, 446 KB)
image.png (1×606 px, 597 KB)

Change 784306 had a related patch set uploaded (by Sbisson; author: Sbisson):

[mediawiki/extensions/Wikistories@master] Add space between stories titles and CTA

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

SBisson added a subscriber: SGautam_WMF.

[...]it makes it seem the padding between a story and the CTA to create a story is insufficient.

I make a quick fix (patch in code review) but I feel like this one would need some more fine tuning of the stories thumbnails and titles. Including trying to put at least 2 lines of the titles.

I would like to keep further refinement for when @SGautam_WMF has access to the app on beta.

Change 784306 merged by jenkins-bot:

[mediawiki/extensions/Wikistories@master] Add space between stories titles and CTA

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

Testing:

This looks better. The padding has been increased. Thanks.

image.png (1×804 px, 236 KB)