Page MenuHomePhabricator

Bring Vector's appearance menu radio markup in sync with latest CdxRadio
Closed, ResolvedPublic1 Estimated Story Points

Description

Background

The markup got out of sync over the many months since the appearance menu got started. With the upcoming changes for font modes this lead to line-height discrepancies relying on just a sub-par of classes for CdxRadio styling.

Goal

Make appearance radio buttons follow CdxRadio markup

Acceptance criteria for done

  • Radios shouldn't be weirdly affected by upcoming font size change (e.g. no vertical misalignment radio button and label)

Event Timeline

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

[mediawiki/skins/Vector@master] clientPreferences: Make appearance radio buttons follow CdxRadio markup

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

Change #1131848 merged by jenkins-bot:

[mediawiki/skins/Vector@master] clientPreferences: Make appearance buttons follow current Codex markup

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

Volker_E updated the task description. (Show Details)