Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#29526 closed defect (bug) (fixed)

wpviews: embedURL: double paragraphs

Reported by: iseulde's profile iseulde Owned by: azaozz's profile azaozz
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.0
Component: TinyMCE Keywords: has-patch
Focuses: javascript Cc:

Description

The view "text" for embedURL views includes paragraphs and when view are post processed, the view "text" is wrapped in a paragraph before replacing the view with it. Result: <p><p>http://...</p></p>. This is then undone by switchEditors.pre_wpautop. To see it for yourself, compare tinyMCE.activeEditor.getContent(); and window.switchEditors.pre_wpautop( tinyMCE.activeEditor.getContent() );. To reproduce you shouldn't paste the url in *visual* editor, because then the "text" will be just the URL. Instead, paste it in the text editor or save the post.

The view "text" should be the URL without anything around it.

Attachments (1)

29526.patch (763 bytes) - added by iseulde 9 years ago.

Download all attachments as: .zip

Change History (5)

#1 @Viper007Bond
9 years ago

  • Component changed from Embeds to Editor

#2 @iseulde
9 years ago

  • Component changed from Editor to TinyMCE

@iseulde
9 years ago

#3 @iseulde
9 years ago

  • Keywords has-patch added; needs-patch removed
  • Milestone changed from Awaiting Review to 4.2

#4 @azaozz
9 years ago

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

In 31485:

TinyMCE wpView: don't insert nested paragraphs when inserting embeddable URLs. Props iseulde, fixes #29526.

Note: See TracTickets for help on using tickets.