WordPress.org

Make WordPress Core

Opened 22 months ago

Last modified 10 months ago

#25785 new defect (bug)

Unexpected Paragraph Formatting Within a Div Container

Reported by: EMG Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.7
Component: Formatting Keywords: needs-patch wpautop needs-unit-tests
Focuses: Cc:

Description

Though HTML mode in WordPress allows for the user to add in their own CSS styling and HTML content formatting if necessary, its normal behavior also makes it auto-format paragraph breaks without the user's input of paragraph tags. This is EXPECTED behavior.

I have discovered that in this specific manner of formatting when writing a post or page (note the LACK of spacing between the div tag and the content and the multiple paragraphs):

<div>Insert FIRST PARAGRAPH here.

Insert SECOND PARAGRAPH here.</div>

The end result (both visually and looking at the source code) is that there is a MISSING opening paragraph tag at the beginning of the first paragraph right after the opening div tag.

IF the user adds their own spacing between the divs and the content, paragraph formatting with happen on its own, but if the user doesn't add in the spacing and starts the paragraphed content right after the div like in the example above, the first paragraph tag isn't automatically added in.

For a screenshot of the end result source code that is generated (with the missing opening paragraph tag), please see the attached file.

Cheers!

Attachments (2)

WP-3-7-ParaTagMissing.png (54.1 KB) - added by EMG 22 months ago.
screenshot of the generated source code with the paragraph formatting error
25785.diff (672 bytes) - added by jorbin 10 months ago.
Unit Tests from MikeHansenMe originally from #30142

Download all attachments as: .zip

Change History (7)

@EMG22 months ago

screenshot of the generated source code with the paragraph formatting error

comment:1 @SergeyBiryukov22 months ago

  • Component changed from Post Formats to Formatting

Related: #849, #3007, #3833, #4857.

comment:2 @nacin20 months ago

  • Keywords wpautop added

comment:3 @nacin20 months ago

  • Keywords needs-unit-tests added

This appears to be a duplicate of #3833 and/or others. Let's get a unit test.

comment:4 @nacin20 months ago

  • Milestone changed from Awaiting Review to Future Release

@jorbin10 months ago

Unit Tests from MikeHansenMe originally from #30142

comment:5 @jorbin10 months ago

MikeHansenMe wrote some unit tests on #30142 for this. The above patch was originally written by him.

Note: See TracTickets for help on using tickets.