Quoth SaveTabView.js, line 81:

self.revertTab.set({hidden: storage_type === 'panelizer_default'});

This does not take into account the fact that the Revert tab may not exist at all -- if I remember correctly, we decided to remove it during the layout sprint in April. The blithe assumption made by this code can cause JS errors, and throw the AJAX system into an uncertain state (jQuery.active might be 1 when it should be 0, among other possible wonkiness).

Proposed solution: check for self.revertTab before calling methods on it.

CommentFileSizeAuthor
#2 2908917-2.patch588 bytesphenaproxima
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima created an issue. See original summary.

phenaproxima’s picture

Status: Active » Needs review
FileSize
588 bytes

DamienMcKenna’s picture

Status: Needs review » Fixed
Parent issue: » #2896009: Plan for Panelizer 8.x-4.1 release

Committed. Thanks!

Status: Fixed » Closed (fixed)

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