Once this CTools issue (#813754: Ability to set variant machine name in Panels UI) is committed, we will have the ability to customize our page manager variant machine names. That's great for creating new variants, but for existing variants some code is needed to allow us to edit them.

CommentFileSizeAuthor
#4 panelizer-n2410363-4.patch2.14 KBdamienmckenna
#1 panelizer-edit-machine-names-2410363-1.patch2.14 KBmrjmd

Comments

mrjmd’s picture

mrjmd commented
Parent issue: » #813754: Ability to set variant machine name in Panels UI
StatusFileSize
new panelizer-edit-machine-names-2410363-1.patch2.14 KB

Patch. Initial testing went smoothly.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: #813754: Ability to set variant machine name in Panels UI » #2155813: Plan for Panelizer 7.x-3.2 release
Related issues: +#813754: Ability to set variant machine name in Panels UI
michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented
Status: Needs review » Reviewed & tested by the community

I tested this per the instructions in the CTools issue and it works great!

Unlike the other two patches, this one applied cleanly with no offset so no new patch needed.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new panelizer-n2410363-4.patch2.14 KB

A minor tweak to fix a minor code formatting thing (missing space between "if" and the parenthesis).

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Needs work

The last submitted patch, 4: panelizer-n2410363-4.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Fixed

Maybe I should have waited for it to finish testing before doing the git-push ;)

Status: Fixed » Closed (fixed)

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