Panelizer should be under admin/structure rather than admin/config.

Comments

DamienMcKenna’s picture

Status: Active » Needs review
FileSize
7.32 KB

"Does exactly what it says on the tin."

merlinofchaos’s picture

Status: Needs review » Needs work

Unfortunately this changes the position of items and causes problems with the 'list' function if you turn on panel choice. I tried to fix that but I seem to be missing something important and I need to move on to other tasks.

DamienMcKenna’s picture

Version: 7.x-2.x-dev » 7.x-3.x-dev
Issue summary: View changes

Needs to be rerolled too.

lpalgarvio’s picture

Status: Needs work » Needs review
FileSize
9.46 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch panelizer_proper_menu.patch. Unable to apply patch. See the log in the details link for more information. View

my version of the patch for panelizer 7.x-3.1

Status: Needs review » Needs work

The last submitted patch, 4: panelizer_proper_menu.patch, failed testing.

lpalgarvio’s picture

FileSize
11.4 KB

updated patch to include changes for arguments (decrease 1 for each page, title, etc argument)

mglaman’s picture

Status: Needs work » Needs review
FileSize
8.71 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch relocate_panelizer_menu-1493296-7.patch. Unable to apply patch. See the log in the details link for more information. View

Patch from #6 didn't apply for me. Here is re-roll that seemed to work find, no breadcrumb issues or anything like that.

Status: Needs review » Needs work

The last submitted patch, 7: relocate_panelizer_menu-1493296-7.patch, failed testing.

DamienMcKenna’s picture

Issue tags: +Needs reroll
DamienMcKenna’s picture

FileSize
11.18 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View

Rerolled, and I updated the README.txt and caught another instance of the old URL.

DamienMcKenna’s picture

Issue tags: +SprintWeekend2015
DamienMcKenna’s picture

Status: Needs work » Needs review
DamienMcKenna’s picture

I need to either commit this or forget about it entirely.

DamienMcKenna’s picture

Issue tags: -Needs reroll
DamienMcKenna’s picture

I'm working on a change notice for this: https://www.drupal.org/node/2409381

DamienMcKenna’s picture

Needs work - the subpages are returning a 403 error.

DamienMcKenna’s picture

Status: Needs review » Needs work
DamienMcKenna’s picture

Status: Needs work » Needs review
FileSize
13.02 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es). View

This version fixes the menu paths for the main config entity subpages under admin/structure/panelizer.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed! Woo-hoo!

  • DamienMcKenna committed 2c275d6 on 7.x-3.x
    Issue #1493296 by DamienMcKenna: Moved the main configuration page.
    

  • DamienMcKenna committed 0de6913 on 7.x-3.x
    Issue #1493296 by DamienMcKenna, mglaman, lpalgarvio: Moved the main...

Status: Fixed » Closed (fixed)

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