Make WordPress Core

Opened 3 years ago

Last modified 2 months ago

#50418 new defect (bug)

Twenty Twenty: Inline Images displaying as block

Reported by: mrfoxtalbot's profile mrfoxtalbot Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.4.2
Component: Bundled Theme Keywords: has-patch needs-testing needs-refresh 2nd-opinion
Focuses: css Cc:

Description

Inline images (those added inside a paragraph block) will display on top of each other (as blocks) instead of next to each other (inline or inline-block).

https://d.pr/i/13DtBl

This will not happen in the editor, only on the public page/post:

https://d.pr/i/vkV9vC

I tried to reproduce this with several themes such as Rockwell, Brandsbury and Coutoire, but they all seem to work fine.

Attachments (6)

inline_images_Twenty_Twenty_bug_Screen Shot 2020-06-18 at 01.32.40.png (40.6 KB) - added by mrfoxtalbot 3 years ago.
Inline Images displaying as block on TwentyTwenty
inline_images_Twenty_Twenty_bug_Screen Shot 2020-06-18 at 01.31.46.png (34.9 KB) - added by mrfoxtalbot 3 years ago.
Inline Images displaying inline in the editor
50418.diff (373 bytes) - added by thimalw 3 years ago.
Set display: inline-block for images inside paragraphs
50418.2.diff (448 bytes) - added by aezazshekh 3 months ago.
Added Patch
50418.3.diff (442 bytes) - added by viralsampat 2 months ago.
I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.
CleanShot 2023-03-24 at 14.54.33@2x.png (452.6 KB) - added by viralsampat 2 months ago.

Download all attachments as: .zip

Change History (14)

@mrfoxtalbot
3 years ago

Inline Images displaying as block on TwentyTwenty

@mrfoxtalbot
3 years ago

Inline Images displaying inline in the editor

#1 @SergeyBiryukov
3 years ago

  • Component changed from Themes to Bundled Theme
  • Summary changed from Inline Images displaying as block on TwentyTwenty to Twenty Twenty: Inline Images displaying as block

@thimalw
3 years ago

Set display: inline-block for images inside paragraphs

#2 @thimalw
3 years ago

My first time submitting a patch. Please let me know if I've done anything wrong.

#3 @thimalw
3 years ago

  • Keywords has-patch needs-testing added

#4 @sabernhardt
2 years ago

  • Focuses css added
  • Keywords needs-refresh 2nd-opinion added
  • Milestone changed from Awaiting Review to Future Release

@thimalw Thanks for contributing a patch! Because Twenty Twenty has mirrored versions of stylesheets for right-to-left languages, the extra selector would need to be added in the style-rtl.css file as well. Then you can make a patch with both files together.

Regarding the selector, I'm unsure whether it should include the .entry-content part. If someone has defined one of the following types of styles (either in a child theme or in the customizer's Additional CSS), the more specific .entry-content p img would override that.

/* case: <p class="hidden-images"><img /></p> */
/* or case: <div class="hidden-images"><p><img /></p></div> */
.hidden-images img {
	display: none;
}
/* case: <p><img class="hidden-image" /></p> */
.hidden-image {
	display: none;
}

Of course, these are likely rare situations, but it can happen.

I think this could be safer, even though it also applies to any images in paragraph tags outside the post content area.

p img,
.entry-content li img {
	display: inline-block;
}

#5 @talldanwp
2 years ago

I noticed a similar issue is happening in Twenty Twenty One (#52287), though in both the editor and front-end.

Last edited 2 years ago by SergeyBiryukov (previous) (diff)

#6 @sabernhardt
3 months ago

@aezazshekh also reported the same in the Verse block on #57912.

#7 @sabernhardt
3 months ago

#57912 was marked as a duplicate.

@aezazshekh
3 months ago

Added Patch

#8 @viralsampat
2 months ago

Hello Team,

I have checked the below-mentioned themes from my end and found that the issue is happening in the "Twenty Twenty" theme.

I have checked this issue on these configuration:

WordPress version: 6.1.1, Gutenberg Version 15.4.0, Theme active: Twenty Twenty,
Browser: Google chrome, Version 111.0.5563.110 (Official Build) (arm64)
Device: MacBook Air M1
OS: macOS 13.1

twentynineteen
twentytwenty
twentytwentyone
twentytwentytwo
twentytwentythree

Thanks,

Last edited 2 months ago by viralsampat (previous) (diff)

@viralsampat
2 months ago

I have checked above mentioned issue and I have resolved it and added patch. Also, I have attached screenshot after resolved this issue.

Note: See TracTickets for help on using tickets.