Headings in mobile VE are missing padding
Open, Needs TriagePublic
Actions

Assigned To

None

Authored By

	Esanders
	Apr 15 2026, 3:22 PM

Project Tags

Referenced Files

None

Subscribers

Aklapper

Esanders

Jdlrobson-WMF

Description

Probably a regression from T414882, padding is now applied to the wrapper classes in Minerva. These wrapper classes don't exist in VE.

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	MWHeadingNode: Add mw-heading* classes	mediawiki/extensions/VisualEditor	master	+31 -0

Customize query in gerrit

Related Objects

Mentioned In: T414882: Additional top margin for Parsoid outputs for sections with no lead
Mentioned Here: T414882: Additional top margin for Parsoid outputs for sections with no lead

Event Timeline

Esanders created this task.Apr 15 2026, 3:22 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 15 2026, 3:22 PM

Esanders mentioned this in T414882: Additional top margin for Parsoid outputs for sections with no lead.Apr 15 2026, 3:23 PM

Let's add a visual regression test for edit mode to https://pixel.wmcloud.org/ to ensure this doesn't happen again. The existing test for DiscussionTools helped us avoid an issue on talk pages with the same change.

Esanders awarded a token.Apr 15 2026, 5:05 PM

Change #1271821 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] MWHeadingNode: Add mw-heading* classes

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

gerritbot added a project: Patch-For-Review.Apr 15 2026, 5:09 PM

Change #1271821 merged by jenkins-bot: