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

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new panelizer-n1493296.patch7.32 KB

"Does exactly what it says on the tin."

merlinofchaos’s picture

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 TN, USA
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 commented
Status: Needs work » Needs review
StatusFileSize
new 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 commented
StatusFileSize
new 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
commented
Status: Needs work » Needs review
StatusFileSize
new 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 TN, USA
commented
Issue tags: +Needs reroll
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new 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 TN, USA
commented
Issue tags: +SprintWeekend2015
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
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 TN, USA
commented
Issue tags: -Needs reroll
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

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

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Needs work - the subpages are returning a 403 error.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new 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 TN, USA
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.