Make WordPress Core

Opened 2 years ago

Last modified 2 months ago

#56143 new defect (bug)

Twenty Nineteen: Separator block alignment issue

Reported by: umesh84's profile umesh84 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 5.0
Component: Bundled Theme Keywords: needs-design-feedback has-patch
Focuses: ui, css Cc:

Description (last modified by sabernhardt)

Twenty Nineteen Themes Separator Block select Default style not center aligned in editor and Front side

Steps to replicate:
1: Activate the Twenty Nineteen Theme
2: add Separator block
3: Select Default style

I have given the video link of the issue below.
https://www.loom.com/share/7f80bd907043491fbe8c28d30dd4cc1a

Attachments (1)

#56143.patch (443 bytes) - added by kajalgohel 2 years ago.
patch added

Download all attachments as: .zip

Change History (5)

@kajalgohel
2 years ago

patch added

#1 @kajalgohel
2 years ago

  • Keywords has-patch added; needs-patch removed

#2 @SergeyBiryukov
2 years ago

  • Summary changed from Themes Twenty Nineteen separator block alignment issue to Twenty Nineteen: Separator block alignment issue

#3 @sabernhardt
2 years ago

  • Description modified (diff)
  • Keywords needs-patch added; has-patch removed
  • Version changed from 6.0 to 5.0

I think aligning the separator line to the side by default is appropriate in Twenty Nineteen because headings have a similar line on the side.

However, if you purposely set it to align center, that does not work at 768 pixels and wider:

@media only screen and (min-width: 768px) {
  .entry .entry-content > *.aligncenter,
  .entry .entry-summary > *.aligncenter,
  .entry .entry-summary > .wp-block-group > .wp-block-group__inner-container > *.aligncenter,
  .entry .entry-content > .wp-block-group > .wp-block-group__inner-container > *.aligncenter {
    margin-left: 0;
    margin-right: 0;
  }
}

The side margins were set to zero before the theme was widely available.

#4 @karmatosed
2 months ago

  • Keywords has-patch added; needs-patch removed
Note: See TracTickets for help on using tickets.