WordPress.org
Get WordPress

Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 19 years ago

#3854 closed defect (bug) (fixed)

text formatting script shouldn't treat scripts as text blocks

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by: rob1n's profile rob1n
Milestone: 2.2 Priority: low
Severity: normal Version: 2.1.1
Component: General Keywords: has-patch commit
Focuses: Cc:

Description

​http://www.ryanjparker.net/fixing-javascript-affiliate-links-in-wordpress-211/

Attachments (2)

3854.diff​ (962 bytes) - added by Nazgul 19 years ago.
3854.2.diff​ (944 bytes) - added by rob1n 19 years ago.
Take out script, object and param

Download all attachments as: .zip

Change History (13)

@Nazgul
19 years ago

#1 @Nazgul
19 years ago

  • Component changed from Administration to General
  • Keywords has-patch added
  • Version set to 2.1.1

#2 @foolswisdom
19 years ago

  • Milestone changed from 2.1.3 to 2.2

#3 @rob1n
19 years ago

+1. <script> tags can be used inline or as a block, though if it's used inline the output has to be inline, too. But that's up to the user.

#4 @Otto42
19 years ago

The object element has the same problem. It shouldn't get extra spaces added around it because it can be inline or block too. Correct?

#5 @rob1n
19 years ago

Yes. Both <object> and <script> *can* be used as inline elements.

#6 @rob1n
19 years ago

  • Owner changed from anonymous to rob1n

#7 @rob1n
19 years ago

  • Keywords commit added

Patch looks good, too.

@rob1n
19 years ago

Take out script, object and param

#8 @rob1n
19 years ago

I'm just going to fix this for <script>.

#9 @rob1n
19 years ago

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

(In [5223]) Don't autop <script> blocks. Props Nazgul. fixes #3854

#10 @Otto42
19 years ago

This really needs to be fixed for objects too. The current editor has weirdness when you insert code from places like Youtube or google video or what have you.

#11 @rob1n
19 years ago

(In [5262]) Don't treat object and param as block elements in wpautop. fixes #3854

Note: See TracTickets for help on using tickets.