Make WordPress Core

Opened 10 months ago

Last modified 13 days ago

#58547 new defect (bug)

Twenty Seventeen: Calendar block header cells should be centered

Reported by: nkeller15's profile nkeller15 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: trivial Version: 6.2.2
Component: Bundled Theme Keywords: has-patch changes-requested
Focuses: css Cc:

Description

The th elements in the Calendar block are not centered like the rest of the text in the Calendar which looks odd.

Attachments (4)

SCR-20230615-mhev.png (161.5 KB) - added by nkeller15 10 months ago.
Screenshot
58547.patch (564 bytes) - added by nkeller15 10 months ago.
Added new code in the blocks.css file to fix this
2017-calendar-front-before.png (402.7 KB) - added by sabernhardt 10 months ago.
header cells are aligned left only when Calendar block is inside post content area
2017-calendar-front-with-patch.png (402.9 KB) - added by sabernhardt 10 months ago.
the patch corrects the alignment on the front

Download all attachments as: .zip

Change History (6)

@nkeller15
10 months ago

Screenshot

@nkeller15
10 months ago

Added new code in the blocks.css file to fix this

@sabernhardt
10 months ago

header cells are aligned left only when Calendar block is inside post content area

@sabernhardt
10 months ago

the patch corrects the alignment on the front

#1 @sabernhardt
10 months ago

  • Keywords 2nd-opinion added
  • Severity changed from normal to trivial
  • Summary changed from Twenty Seventeen - Calendar Widget <th> alignment should be centered to Twenty Seventeen: Calendar block header cells should be centered

The Calendar block is not as likely to be used inside the post content area as within one of the widget areas in a "classic" theme such as Twenty Seventeen. The header cells already are centered where the block is expected.

That said, the patch works on the front end. If the styles are changed in blocks.css, then editor-blocks.css probably should have the same.

#2 @karmatosed
13 days ago

  • Keywords changes-requested added; 2nd-opinion removed

I would like to see this patch at least refreshed also to have it work for the editor before further consideration. I think there might be a case to add it if it doesn't break anything but benefits in the particular case, that said it needs to be both front and editor applied. Thank you everyone for your collaboration so far.

Note: See TracTickets for help on using tickets.