WordPress.org

Make WordPress Core

Opened 112 minutes ago

Last modified 86 minutes ago

#49699 new defect (bug)

[Twenty Nineteen] Center- and right-aligned heading accents appear broken.

Reported by: kjellr Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: trunk
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

By default, H1 and H2 headings in Twenty Nineteen include a little accent before them:

http://cldup.com/y7fpYMLgyB.png

When those headings are center- and right-aligned however, the accent stays in place:

http://cldup.com/CcoBuLElzG.png

This appears broken.


The most obvious fix would be to have those dividers move around with the text:

http://cldup.com/xFQor-jhYK.png

I'ma little partial to that one for consistency's sake, but another option would be to just remove those accents when the heading is not left aligned:

http://cldup.com/GgdpAkGLnU.png

Attachments (2)

49699.patch (4.9 KB) - added by kjellr 111 minutes ago.
49699-alt.patch (3.0 KB) - added by kjellr 111 minutes ago.

Download all attachments as: .zip

Change History (4)

@kjellr
111 minutes ago

#1 @kjellr
109 minutes ago

49699.patch is a patch for that first option: moving the accent in line with the heading's alignment. (This seems like the most reasonable approach to me.)

But just in case, 49699-alt.patch is a patch for that second one: removing the accent for center and right alignments.

#2 @melchoyce
86 minutes ago

I like moving the accent line depending on alignment! Feels like such a distinctive style for the theme.

Note: See TracTickets for help on using tickets.