WordPress.org

Make WordPress Core

Opened 5 weeks ago

Last modified 3 weeks 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:

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 5 weeks 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 3 weeks ago.

Download all attachments as: .zip

Change History (9)

@laurelfulford
5 weeks 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
5 weeks ago

  • Description modified (diff)

#2 @nielslange
5 weeks ago

Howdy @laurelfulford 👋

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

#3 @laurelfulford
3 weeks 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
3 weeks ago

Thanks, @laurelfulford.

Last edited 3 weeks ago by nielslange (previous) (diff)

@nielslange
3 weeks ago

#5 @nielslange
3 weeks ago

The patch is ready for testing. 😀

Last edited 3 weeks ago by nielslange (previous) (diff)

#6 @nielslange
3 weeks ago

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

#7 @kjellr
3 weeks 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.