Steps to reproduce:
1. Open ctools modal popup several times on the same page (without reloading)
2. Close popup via escape key
3. The subsequent close event (CToolsDetachBehaviors) will be triggered multiple times (one time per each newly opened popup).

It caused by the fact that each time when we call Drupal.CTools.Modal.modalContent() we bind new event, and it does not unbind-ed sufficiently on close event.

CommentFileSizeAuthor
#4 2781009-4.patch581 bytesakashkumar07
#2 7.x-1.9-multiple-close-event-2781009-2.patch1.01 KBIRuslan

Comments

IRuslan created an issue. See original summary.

IRuslan’s picture

IRuslan commented
Status: Needs work » Needs review
StatusFileSize
new 7.x-1.9-multiple-close-event-2781009-2.patch1.01 KB
chris matthews’s picture

chris matthews commented
Status: Needs review » Needs work
Issue tags: +Needs reroll

The 2 year old patch to modal.js does not apply to the latest ctools 7.x-1.x-dev.

Checking patch js/modal.js...
error: while searching for:
      $('body').unbind( 'keypress', modalEventHandler );
      $('body').unbind( 'keydown', modalTabTrapHandler );
      $('.close').unbind('click', modalContentClose);
      $('body').unbind('keypress', modalEventEscapeCloseHandler);
      $(document).trigger('CToolsDetachBehaviors', $('#modalContent'));

      // Set our animation parameters and use them

error: patch failed: js/modal.js:587
error: js/modal.js: patch does not apply
akashkumar07’s picture

akashkumar07 commented
Status: Needs work » Needs review
StatusFileSize
new 2781009-4.patch581 bytes

Reroll of #2.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Fixed
Issue tags: -Needs reroll

I noticed the event is different from when it was bound as well so changed that on commit. Thanks @IRuslan

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.