Page MenuHomePhabricator

Homepage: text style issues in recent questions list of mobile mentor and help module dialogs
Open, LowPublicBug

Description

In the context of the mobile Newcomer homepage, the text indicating the question age in the recent questions list of the Mentorship and Help module details dialog, presents some issues.
According to mockup's specifications, font-size should be .8em and letter-spacing should be 0.

Actual Results:

Expected Results:

.question-posted-on {
  font-size: 0.8em;
  letter-spacing: 0;
}

Event Timeline

Cntlsn created this task.Mon, Aug 12, 5:22 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMon, Aug 12, 5:22 PM
Cntlsn triaged this task as Low priority.Mon, Aug 12, 5:23 PM

You asked for both the font-size: 0.85em and letter-spacing: 0.02em rules here: T218966#5130398.

Should these:

  • A: Stay the way they are?
  • B: Change to 0.8em and 0 on both desktop and mobile?
  • C: Change to 0.8em and 0 on mobile, but stay at 0.85em and 0.02em on desktop?

(If C, why?)

phuedx moved this task from Needs triage to Growth on the Mobile board.Thu, Aug 15, 11:10 AM

I'm answering here to your question, plus questions at T230364 and T230373.

The answer is C and here's why:

  • The request at T218966#5130398 was about trying to simulate the usage of a different font for the light text, and avoid loading another font family with the page. This other font 'Roboto' comes from legacy files by @RHo I started working with when designing the homepage. When I found out I was using a different font I decided to maintain a similar style, since the visual appearance of the lighter Roboto font on desktop actually increased legibility of smaller text.
  • On the mobile module previews the light text has a different use (longer informative text, versus bite sized data on desktop), plus mobile renders fonts slightly differently which was making Roboto break the rhythm of the page. I then decided to rely on system font to increase legibility and rhythm everywhere.
  • Last but not least, to be very precise text at T230373 could potentially be letter-spacing: .02em, but it's good to keep consistency within the platform just as we did for desktop and so make it letter-spacing: 0 like the other occurrences.

I hope this will explain well the decisions made.

RHo added a comment.Fri, Aug 16, 8:26 AM

Apologies if some of the files were in Roboto that caused the confusion, that shouldn't be the case, rather OOUI font stack should be used. It was not intentional but must have occurred since my default typeface settings in Sketch were previously for Android designs.

Change 530572 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/extensions/GrowthExperiments@master] Help/Mentor: update text size of recent question timestamp

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

Change 530572 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Help/Mentor: update text size of recent question timestamp

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

Checked in betalabs - all specs seems to be in place: