Page MenuHomePhabricator

Replace `line-height-heading`'s 1.3 by 1.375
Closed, ResolvedPublic2 Estimated Story Points

Description

Background

Coming from newly established Codex Figma font definitions, and in relations to parent tasks, we should settle on 1.375 for headings' line-height across products instead of current 1.3.

CodeSearch: https://codesearch-beta.wmcloud.org/search/?q=%40line-height-heading&files=&excludeFiles=&repos=

Example in legacy Vector

beforewith 1.375
image.png (584×2 px, 277 KB)
image.png (592×2 px, 279 KB)

Example in Vector 2022

beforewith 1.375
image.png (502×2 px, 144 KB)
image.png (506×2 px, 145 KB)

We're talking about an increase from 37.44px to 39.6px. This should not cause any side-effects.

Acceptance criteria for done

  • Replace 1.3 by 1.375 in all occurrences

Event Timeline

Change 899617 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/skins/Vector@master] styles: Increase `line-heading-heading` slightly to follow Codex

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

Change 899618 had a related patch set uploaded (by VolkerE; author: VolkerE):

[wikimedia-ui-base@master] Increase `line-height-heading` slightly to follow Codex

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

Change 899618 merged by jenkins-bot:

[wikimedia-ui-base@master] Increase `line-height-heading` slightly to follow Codex

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

Restricted Application triaged this task as High priority. · View Herald TranscriptMar 16 2023, 3:46 PM
Volker_E changed the task status from Open to Stalled.Mar 16 2023, 5:33 PM
Jdlrobson added subscribers: ovasileva, Jdlrobson.

Assuming this is blocked on web team. If so, adding to our backlog to make sure @ovasileva sees this request. Please let me know if I've misunderstood.

@Jdlrobson That's correct. Also brought to attention of @KieranMcCann and @RHo already. Note that this (small) change would also unblock a small technical debt in the Codex tokens application IMO easily resolvable.

LGoto set the point value for this task to 2.Mar 23 2023, 5:38 PM

Change 899617 merged by jenkins-bot:

[mediawiki/skins/Vector@master] styles: Increase `line-heading-heading` slightly to follow Codex

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

This should be testable shortly on the beta cluster.

Jdlrobson changed the task status from Stalled to Open.Mar 29 2023, 1:25 AM

Looks good to me.

@Volker_E Do you want to check this meets your expectations?

@Volker_E please resolve this one if you have what you need, otherwise web team will assume this one is done.

Yes, works perfectly as intended from my POV.