Make WordPress Core

Opened 2 years ago

Last modified 2 years ago

#61723 new defect (bug)

Twenty Sixteen: Image Block Alignment Offset (Left/Right)

Reported by: mrfoxtalbot Owned by:
Priority: normal Milestone: Future Release
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch changes-requested
Cc: Focuses: javascript

Description

After the 6.6 updated, floated images in Twenty Sixteen are offset: margin-left: -40%;

Steps to reproduce

  1. Add a new post through this path: My Site → Posts → Add New
  2. Add texts with "Paragraph" block.
  3. Add an image with "Image" block.
  4. Resize the image, and change alignment to either right or left.
  5. Save and preview to see the issue on the live page or preview.
  6. What you expected to happen
  7. I will see the image alignment correctly on the live/preview page.

What happened

The image is slightly shifted to the left, causing text to overlap with the image when right-aligned, and creating a larger gap between the text and the image when left-aligned.

This was originally reported in ​https://github.com/Automattic/wp-calypso/issues/92833

Attachments (5)

350391848-764f0fc9-6fe2-43d8-9403-33031b4c2d4f.png​ (2.0 MB ) - added by mrfoxtalbot 2 years ago.
Offset Image Floats in Twenty Sixteen
350391896-590de617-be42-40f7-8f63-b4ab1ff8bb0d.png​ (1.9 MB ) - added by mrfoxtalbot 2 years ago.
Offset Image Floats in Twenty Sixteen (right)
SCR-20240722-ofzu.png​ (483.2 KB ) - added by karmatosed 2 years ago.
Showing it working on front.
SCR-20240722-ohfi.png​ (1.3 MB ) - added by karmatosed 2 years ago.
Adding multiples to see if issue.
Screenshot 2024-09-03 at 18.05.51.png​ (344.4 KB ) - added by gikaragia 2 years ago.
Center alignment not respected

Change History (13)

@mrfoxtalbot
2 years ago

Offset Image Floats in Twenty Sixteen

@mrfoxtalbot
2 years ago

Offset Image Floats in Twenty Sixteen (right)

#1 @karmatosed
2 years ago

  • Keywords reporter-feedback added

Unfortunately, using these steps and the latest version of Twenty Sixteen, I am unable to replicate. I will share screenshots but my thoughts are there might be other steps or things that are at play here to consider. Once we have those we can progress knowing them.

@karmatosed
2 years ago

Showing it working on front.

@karmatosed
2 years ago

Adding multiples to see if issue.

#2 @sabernhardt
2 years ago

  • Focuses javascript added
  • Keywords needs-patch added; reporter-feedback removed
  • Milestone Awaiting Review → Future Release
  • Version 6.6

The negative margin itself is part of an ​intentional design feature for full-width images and aligned blockquote elements.

The bug with resized images requires a specific set of conditions to reproduce, and I was able to experience it with WordPress 6.5 and Twenty Sixteen 3.2 as well as WP 6.6.

  • Place the Image block after enough content so that it would appear below the entry meta on large screens.
  • Choose an image that is at least 840 pixels wide.
  • Change the Width setting to resize the image, but keep the "Large" or "Full Size" Resolution setting.

The below entry meta script checks the width attribute for the size, but the editor adjusts the image width in the style attribute.

#3 @sabernhardt
2 years ago

The below-entry-meta class can be added incorrectly whether the block has some alignment setting (left, right or center):

<div class="wp-block-image">
	<figure class="alignleft size-large is-resized">
		<img width="1024" height="512" class="wp-image-30 below-entry-meta" style="object-fit:cover;width:294px;height:auto" ... >
		<figcaption class="wp-element-caption below-entry-meta">caption</figcaption>
	</figure>
</div>

or without alignment:

<figure class="wp-block-image size-large is-resized">
	<img width="1024" height="512" class="wp-image-30 below-entry-meta" style="object-fit:cover;width:294px;height:auto" ... >
	<figcaption class="wp-element-caption below-entry-meta">caption</figcaption>
</figure>
Last edited 2 years ago by sabernhardt (previous) (diff)

This ticket was mentioned in ​PR #7073 on ​WordPress/wordpress-develop by ​@umeshsinghin.


2 years ago
#4

  • Keywords has-patch added; needs-patch removed

#5 @sabernhardt
2 years ago

  • Keywords changes-requested added

I just added the word "incorrectly" in comment:3 because those are examples of how the below-entry-meta class is wrong with the current functions.js. The script needs to avoid adding the class when the style attribute sets a smaller size than the width attribute.

#6 @umeshsinghin
2 years ago

Hi @sabernhardt,

I have made the changes as suggested. Please review and confirm.

#7 @karmatosed
2 years ago

I am going to add testing keyword to this so we can ensure that happens. When testing please note the specific instructions as it needs those to recreate, thank you @sabernhardt.

@gikaragia
2 years ago

Center alignment not respected

#8 @gikaragia
2 years ago

Another related issue caused by the "Overhanging Full-Size Image" feature is that the center alignment is ignored. When the user sets a center alignment to the image block, the image is aligned to the left regardless (see the attachment above)

Shouldn't the setting be respected and the image be centered in the middle?

This was raised initially here: ​https://github.com/Automattic/themes/issues/7336

Note: See TracTickets for help on using tickets.