If you edit a wysiwyg field the popup isn't triggered. Patch following.

CommentFileSizeAuthor
#1 wysiwyg_protection-1276370-1.patch521 bytesidflood
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

idflood’s picture

idflood CreditAttribution: idflood commented
FileSize
wysiwyg_protection-1276370-1.patch521 bytes

Here is a simple patch. It uses "change" event instead of "blur" and adds "textarea" to the listened inputs. If this patch is applied the following issue can be closed: #1158998: usage of change event instead of blur

idflood’s picture

idflood CreditAttribution: idflood commented
Status: Active » Needs review
erykmynn’s picture

erykmynn CreditAttribution: erykmynn commented

I have installed this on my 7x instance and it appears to work fine! It also seems to work with the back button (which was mentioned elsewhere.) FWIW I am NOT using the overlay...

checker’s picture

checker CreditAttribution: checker commented

I tested the patch and for me it does not work. I get always the alert also if nothing has changed. I 'm using WYSIWYG module with CKeditor.

ggevalt’s picture

ggevalt CreditAttribution: ggevalt commented

An added wrinkle... has anyone dealt with this on Drupal 6?
g

czigor’s picture

czigor CreditAttribution: czigor commented

The patch does not work for WYSIWYG module + CKEditor library.
Here, however is a patch that works for this combination:
https://drupal.org/comment/8211153#comment-8211153

czigor’s picture

czigor CreditAttribution: czigor commented
Issue summary: View changes
Status: Needs review » Closed (duplicate)

Duplicate of #1358004: [D7] Add support for CKEditor library.

It works now with CKeditor. For other wysiwyg editors please open a different issue.