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

CommentFileSizeAuthor
#19 panelizer-n1493296-19.patch13.02 KBDamienMcKenna
#11 panelizer-n1493296-11.patch11.18 KBDamienMcKenna
#7 relocate_panelizer_menu-1493296-7.patch8.71 KBmglaman
#6 panelizer_proper_menu2.patch11.4 KBlpalgarvio
#4 panelizer_proper_menu.patch9.46 KBlpalgarvio
#1 panelizer-n1493296.patch7.32 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
panelizer-n1493296.patch7.32 KB

"Does exactly what it says on the tin."

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
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

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Issue summary: View changes

Needs to be rerolled too.

lpalgarvio’s picture

lpalgarvio CreditAttribution: lpalgarvio commented
Status: Needs work » Needs review
FileSize
panelizer_proper_menu.patch9.46 KB

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

lpalgarvio CreditAttribution: lpalgarvio commented
FileSize
panelizer_proper_menu2.patch11.4 KB

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

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented
Status: Needs work » Needs review
FileSize
relocate_panelizer_menu-1493296-7.patch8.71 KB

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

jeffdiecks queued 7: relocate_panelizer_menu-1493296-7.patch for re-testing.

Status: Needs review » Needs work

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Issue tags: +Needs reroll
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
FileSize
panelizer-n1493296-11.patch11.18 KB

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Issue tags: +SprintWeekend2015
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2155813: Plan for Panelizer 7.x-3.2 release

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Issue tags: -Needs reroll
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Needs work - the subpages are returning a 403 error.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Needs work
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review
FileSize
panelizer-n1493296-19.patch13.02 KB

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

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
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.