Making WordPress.org

Opened 5 years ago

Closed 5 years ago

#4870 closed defect (bug) (fixed)

Theme is missing proper image left or right alignment styling

Reported by: bph's profile bph Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: HelpHub (wordpress.org/documentation) Keywords: needs-patch
Cc:

Description

Earlier today I tried to add an image to a text on an article. It worked fine in the editor as you can see in this screenshot:

In the preview in didn't quite translate it. It was right aligned but on its own 'plane' above the text. Here another screenshot

Attachments (3)

Screen Shot 2019-11-23 at 12.23.46 PM.png (59.6 KB) - added by bph 5 years ago.
Image right aligned in editor next to text
Screen Shot 2019-11-23 at 12.23.37 PM.png (58.8 KB) - added by bph 5 years ago.
Preview of right aligned image
_helphub.scss (1.2 KB) - added by audrasjb 5 years ago.
Patch directly made on the file, since HelpHub repo is not synchronized.

Download all attachments as: .zip

Change History (9)

@bph
5 years ago

Image right aligned in editor next to text

@bph
5 years ago

Preview of right aligned image

This ticket was mentioned in Slack in #docs by bph. View the logs.


5 years ago

#2 @bph
5 years ago

  • Keywords needs-design added
  • Summary changed from HelpHub Theme is missing proper image left or right alignment styling to Theme is missing proper image left or right alignment styling

#3 @bph
5 years ago

pardon the quirky screenshot references - n00by at work.

#4 @audrasjb
5 years ago

  • Keywords needs-patch added; needs-design removed

Hi,

Thanks for ticket. I can confirm the issue, this is quite uncomfortable for writing documentation.

The issue could be fixed quite easily, just need to remove this CSS rule:

.helphub-page .entry-content section .container .wp-block-image {
    overflow: hidden;
}

Cheers,
Jb

@audrasjb
5 years ago

Patch directly made on the file, since HelpHub repo is not synchronized.

#5 @audrasjb
5 years ago

So,

We need to remove this CSS rule:

.helphub-page .entry-content section .container .wp-block-image {
    overflow: hidden;
}

from https://wordpress.org/support/wp-content/themes/pub/wporg-support/sass/site/_helphub.scss

But it doesn’t exists on GitHub file: https://github.com/WordPress/HelpHub/blob/master/themes/wporg-support/sass/site/_helphub.scss

I added a patch, above this comment.

Cheers,
Jb

#6 @coffee2code
5 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 9536:

Support Theme: Fix flow of text around a right-aligned image block.

Props bph, audrasjb.
Fixes #4870.

Note: See TracTickets for help on using tickets.