Opened 22 months ago

Closed 6 months ago

Last modified 6 months ago

#18279 closed defect (bug) (worksforme)

Using Featured image: post state is set dirty, even when no changes to content

Reported by: jacobcoens Owned by:
Priority: normal Milestone:
Component: Post Thumbnails Version: 3.2.1
Severity: normal Keywords: has-patch
Cc:

Description

Browser: IE

Behavior:
When editing a post and clicking 'Set featured image', a (thickbox) popup opens. When cancelling, the MCE state is dirty, even when no changes to the content have been made.
This is causing the message 'Are you sure you want to navigate away from this page?' to be displayed when navigating to another page.

Cause:
This happens in IE because a bookmark is created to store the caret position.

Fix:
Remember the 'dirty' state of MCE, and re-apply it to the editor after creating the bookmark.

In wp-admin/js/media-upload.dev.js, replace line 62-68 with below:

var ed;

if ( typeof tinyMCE != 'undefined' && tinymce.isIE && ( ed = tinyMCE.activeEditor ) && !ed.isHidden() ) {

var dirty = ed.isDirty();
ed.focus();
ed.windowManager.insertimagebookmark = ed.selection.getBookmark();
ed.isNotDirty = !dirty;

}

Attachments (1)

18279.patch (527 bytes) - added by SergeyBiryukov 22 months ago.

Download all attachments as: .zip

Change History (5)

  • Keywords has-patch removed
  • Keywords has-patch added
  • Keywords close added

Can we close this ticket in favour of the new media modal?

  • Keywords close removed
  • Milestone Awaiting Review deleted
  • Resolution set to worksforme
  • Status changed from new to closed

Could not reproduce the original bug using IE 8 neither in 3.2.1 (reported version) nor in 3.5-beta3.

Last edited 6 months ago by SergeyBiryukov (previous) (diff)
Note: See TracTickets for help on using tickets.