WordPress.org

Make WordPress Core

Opened 11 months ago

Last modified 11 months ago

#45912 assigned enhancement

Twenty Nineteen: Horizontal rule is very narrow

Reported by: laurelfulford Owned by: nielslange
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-screenshots good-first-bug has-patch
Focuses: Cc:
PR Number:

Description (last modified by laurelfulford)

Originally reported by @joyously in the Twenty Nineteen GitHub repo:

The <hr> tag outputs a line that is very narrow (max-width: 2.25em), and seems like it would be an unexpected style in legacy content.

It's matches the current default style for the separator block, but it's possible to change that style to be full-width, or centred dots; for the <hr> tag it can't be changed.

For the purpose of preserving the expected visual appearance in legacy content, let's switch its appearance with that of the wide separator block style.

Original ticket here: https://github.com/WordPress/twentynineteen/issues/83

Attachments (2)

83-horizontal-rule.png (5.9 KB) - added by laurelfulford 11 months ago.
Twenty Nineteen three separator block variants. hr tags currently look like the first one, but should be updated to look like the second one.
45912.diff (23.4 KB) - added by nielslange 11 months ago.

Download all attachments as: .zip

Change History (9)

@laurelfulford
11 months ago

Twenty Nineteen three separator block variants. hr tags currently look like the first one, but should be updated to look like the second one.

#1 @laurelfulford
11 months ago

  • Description modified (diff)

#2 @nielslange
11 months ago

Howdy @laurelfulford 👋

I’ld love to provide a patch for this issue the next days! 😉

#3 @laurelfulford
11 months ago

  • Milestone changed from Awaiting Review to Future Release
  • Owner set to nielslange
  • Status changed from new to assigned

Thanks @nielslange!

I'll assign this to you for now, so it's known that it's "taken" :)

#4 @nielslange
11 months ago

Thanks, @laurelfulford.

Last edited 11 months ago by nielslange (previous) (diff)

@nielslange
11 months ago

#5 @nielslange
11 months ago

The patch is ready for testing. 😀

Last edited 11 months ago by nielslange (previous) (diff)

#6 @nielslange
11 months ago

@laurelfulford, @allancole or @kjellr: Anyone up for a quick test? 😛

#7 @kjellr
11 months ago

  • Keywords has-patch added; needs-patch removed

This works for me!

Before:
https://cldup.com/FQCaVSip_t-3000x3000.png

After:
https://cldup.com/1sT08PEfSO-3000x3000.png

Thanks, @nielslange!

Note: See TracTickets for help on using tickets.