WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#16388 closed defect (bug) (invalid)

Unresponsive script when switching from Visual to HTML Editor with long posts and lots of HTML

Reported by: yoavf Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Editor Keywords:
Focuses: Cc:

Description (last modified by yoavf)

When editing long posts with a fair amount of (nested) HTML tags, browser will issue an unresponsive script error when switching from Visual to HTML editor.

Attached are two very long posts, similar except testcase-unresponsive.txt has about 2500 extra

<p><br class="spacer_" /></p>

I haven't encountered any problems with testcase-ok.txt.

Hoping there's something that can be optimized here.

Attachments (1)

testcases.zip (23.6 KB) - added by yoavf 7 years ago.

Download all attachments as: .zip

Change History (6)

@yoavf
7 years ago

#1 @yoavf
7 years ago

  • Description modified (diff)

#2 @buch0090
7 years ago

FYI, the script error doesn't happen with earlier version of WP (WPMU 2.6.5). This tells me that tinymce/wp added some extra regex's (or something).

#3 @azaozz
7 years ago

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

The <br class="spacer_" /> placeholder actually comes from the TinyMCE Advanced plugin and can be disabled there. Also it's not used any more in the updated version that is compatible with 3.1 (currently in beta).

Closing as it seems to be plugin issue.

#4 @nacin
7 years ago

  • Milestone Future Release deleted

#5 @buch0090
7 years ago

From what I can tell, this is not a plugin issue. If I remove all the class="spacer_" I still get the JS error. This is with the tinyMCE Advanced completely removed.

I did two tests...first removed the class="spacer_" then removed all <BR>'s and still get the error.

Note: See TracTickets for help on using tickets.