Page MenuHomePhabricator

Homepage: mentor text style issues in mobile Mentorship module dialog
Closed, ResolvedPublicBUG REPORT

Description

In the context of the mobile Newcomer homepage, mentor info text in the Mentorship module dialog has styling issues.

Actual Results:

Screenshot 2019-08-06 17.57.45.png (314×750 px, 38 KB)

letter-spacing is 0.02em

Expected Results:

Screenshot 2019-08-12 19.02.51.png (226×750 px, 29 KB)

letter-spacing should be 0

.growthexperiments-homepage-mentorship-editcount, .growthexperiments-homepage-mentorship-lastactive {
  letter-spacing: 0;
}

Event Timeline

There's a letter-spacing: 0.02em rule on these because you asked for it here: T218966#5130398

Should the increased letter spacing continue to be applied on desktop, and only be undone on mobile, or should it be removed completely?

Change 530581 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/extensions/GrowthExperiments@master] Mentorship: update mentor edit count and last active text

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

Change 530581 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Mentorship: update mentor edit count and last active text

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

There's a letter-spacing: 0.02em rule on these because you asked for it here: T218966#5130398

Should the increased letter spacing continue to be applied on desktop, and only be undone on mobile, or should it be removed completely?

Hi @SBisson @Catrope - can we remove letter-spacing from all the modules please? It has very marginal benefit considering we are using the standard sans-serif font stack anyway, and in the different languages we have the module, I prefer to not change the letter-spacing from the default of 0 in case of unintended visual effects.

Change 531495 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/extensions/GrowthExperiments@master] Use only default letter spacing

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

Change 531495 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Use only default letter spacing

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

Checked - letter-spacing is removed.

Screen Shot 2019-08-23 at 11.17.09 AM.png (802×440 px, 88 KB)