I discovered a line of code in panels_ipe.js which seems to be for debugging.
Guess this line should have never made it into a release...

Currently in panels_ips.js (Line 5):

// randomly lock a pane.
// @debug only
Drupal.settings.Panels = Drupal.settings.Panels || {};
Drupal.settings.Panels.RegionLock = {
  10: { 'top': false, 'left': true, 'middle': true }
}

I think it should be:

Drupal.settings.Panels = Drupal.settings.Panels || {};
CommentFileSizeAuthor
#1 panels-ipe-remove-debug-2405733-1.patch481 bytesdsnopek

Comments

dsnopek’s picture

Version: 7.x-3.4 » 8.x-3.x-dev
Status: Active » Needs review
StatusFileSize
new481 bytes

Wow, we really need to remove this!

I was just debugging an issue for a client and they were wondering "why can't we move this pane to any other region?" It turns out it has a pid of 10, which is what the debug code is using. :-/

Here's a patch that fixes this for the client site.

dsnopek’s picture

Version: 8.x-3.x-dev » 7.x-3.x-dev

Oops! I meant 7.x-3.x.

  • japerry committed ed96710 on 7.x-3.x authored by dsnopek
    Issue #2405733 by dsnopek: Debug Code in panels_ipe.js
    
japerry’s picture

Status: Needs review » Fixed

thanks! yup thats easy, fixed.

dsnopek’s picture

Thanks!

Status: Fixed » Closed (fixed)

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