Panels desperately needs the ability to have the flexible layout system to be reusable. This means creating a layout via flexible and giving it a name and having it be exportable. Patch forthcoming.

Comments

merlinofchaos’s picture

Status: Active » Needs review
StatusFileSize
new23.49 KB

Here's a patch for review. This will go in soon but I'm hoping there are some eyeballs to catch obvious errors.

Not yet done:

1) I just realized that I haven't added the checkbox "make this reusable" like custom content for CTools has. I'll add this next patch.
2) There's a couple of additional flexible features/cleanup I need to do. That may be separate. Particularly, finer control on classes, plus checking to see if we really *need* to render the left/middle/right divs inside a column and not rendering them if so. (we may also be able to simply avoid rendering column/row divs if columns and rows are, in fact, alone. Plus I want an option to fix the canvas width, and another option to disable the 99% scaling. These probably should be a separate patch but are part of this effort.

merlinofchaos’s picture

Oh one other comment: Preview is lame right now because there's no content to preview it with. Need to make sure that provides custom content.

merlinofchaos’s picture

StatusFileSize
new28.4 KB

Updated patch allows flexible layouts to be made reusable in the layout designer.

This one turned out not quite as nice as I would've liked, but it works; the workflow is different and I am sure that will confuse people, but the option to embed a form within a form is not there, and flexible has no visiblity into the actual panels edit form.

merlinofchaos’s picture

Oh yeah. Still needs help to make the preview work.

merlinofchaos’s picture

StatusFileSize
new28.4 KB

Ok the preview button works in this patch

realityloop’s picture

Status: Needs review » Needs work

I found the process of selecting a saved layout when creating new panels logical.

Seems to work well, when viewing the front end version of the panel and using the "Customize this page" method of editing the panel there is no option to add node parts as you can from admin/build/pages editor.

Also there is currently no way to add content to regions that don't already contain some form of content when using "Customize this page", all I get are the save and cancel buttons.

I'd also suggest changing the checkbox title:
Do not switch this panel
to something like
Keep current panel layout flexible

merlinofchaos’s picture

Those seem to all be issues with the IPE, which is pretty new in -dev and definitely not ready for prime time. There is probably no option to add node parts because there's no node context available at that point. I'll have to take a look at that when I move on to the IPE later.

I like your wording for the checkbox more, definitely, and will switch to that.

merlinofchaos’s picture

Status: Needs work » Fixed

Ok, I fixed fixed the preview, cleaned up some code and committed this!

realityloop’s picture

Cool!

Status: Fixed » Closed (fixed)

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