WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 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:
Priority: normal Milestone:
Component: Editor Version: 3.1
Severity: normal Keywords:
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 2 years ago.

Download all attachments as: .zip

Change History (6)

yoavf2 years ago

comment:1 yoavf2 years ago

  • Description modified (diff)

comment:2 buch00902 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).

comment:3 azaozz2 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.

comment:4 nacin2 years ago

  • Milestone Future Release deleted

comment:5 buch00902 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.