Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#24755 closed defect (bug) (fixed)

Multiple copies of backup-post-in-browser-different message

Reported by: markjaquith's profile markjaquith Owned by:
Milestone: 3.6 Priority: high
Severity: critical Version: 3.6
Component: Autosave Keywords:
Focuses: Cc:



At most, this message should show up once.

Attachments (1)

24755.patch (516 bytes) - added by azaozz 11 years ago.

Download all attachments as: .zip

Change History (5)

#1 @azaozz
11 years ago

Can't reproduce either. There is something weird going on at that site. The call to the function that displays the error message is hard-coded (see here and here). Not sure how it can run multiple times unless the actual autosave.js file is loaded multiple times.

Looking at the screencast there is a plugin (or several plugins) extending TinyMCE. May have something to do with this.

#2 @azaozz
11 years ago

Actually multiple copies of the error message will be inserted if there are more than one <h2> at the top of the page. Can reproduce when there are 14 <h2>'s and only the first is visible.

Version 1, edited 11 years ago by azaozz (previous) (next) (diff)

11 years ago

#3 @markjaquith
11 years ago

In 24710:

Only add the autosave notice to the first <h2> on the page.

Props azaozz. See #24755.

#4 @nacin
11 years ago

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

Closing as fixed, pending further forum feedback.

Note: See TracTickets for help on using tickets.