Page MenuHomePhorge

Add flexible line height to remarkup headers
ClosedPublic

Authored by bekay on Jan 22 2024, 16:57.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 22:38
Unknown Object (File)
Wed, Apr 24, 22:23
Unknown Object (File)
Wed, Apr 24, 21:42
Unknown Object (File)
Wed, Apr 17, 18:21
Unknown Object (File)
Wed, Apr 17, 06:54
Unknown Object (File)
Sat, Mar 30, 12:07
Unknown Object (File)
Sat, Mar 30, 05:40
Unknown Object (File)
Fri, Mar 29, 19:36

Details

Summary

The text in remarkup has a fixed line height. The headers will have the same fixed line height, even if their font size can be bigger than this line height. Especially on mobile this results in ugly sticked together lines which are less readable.

This revision will give the header a flexible line height of 1.3. Unitless means relative to the font size.

BeforeAfter
image.png (201×310 px, 14 KB)
image.png (258×309 px, 16 KB)

(I have moved the same margin attribute for every h{2-6} into the new declaration.)

Close T15716.

Test Plan

Just checkout and look in wiki articles with headers to see if everything looks nice and neat.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bekay requested review of this revision.Jan 22 2024, 16:57

I'm inclined to merge this. Any objections?

This revision is now accepted and ready to land.Jan 26 2024, 05:06