Could we allow for a default renderer pipeline for panels? (Allow option to specify default IPE/Standard/etc...)

CommentFileSizeAuthor
#4 1836716-4-panels-renderer-pipeline_default.patch1.45 KBgrndlvl
#1 1836716-1-panels-renderer-pipeline_default.patch1.37 KBgrndlvl

Comments

grndlvl’s picture

grndlvl commented
StatusFileSize
new 1836716-1-panels-renderer-pipeline_default.patch1.37 KB

Here is my first pass at it. I think ideally it would be preferable to set this default per panel type (Not sure what the Panel term would be for this it is noted as just "module_name" on forms as 'panels_common_settings'). I was able to add this setting appropriately, but I was having an issue figuring out where I can apply it to the handler->conf array.

For now I have rolled the patch as a global setting that sets the default for the pipeline for all panel types(This works for our current case). Any insight would be helpful.

Thanks,

Jonathan

grndlvl’s picture

grndlvl commented
Status: Active » Needs review
merlinofchaos’s picture

merlinofchaos commented

Oh this is nifty.

The new menu item definitely needs a description. Something like, "The default renderer for new panel pages" since it won't affect any pre-existing page, and that's an important UI note.

grndlvl’s picture

grndlvl commented
StatusFileSize
new 1836716-4-panels-renderer-pipeline_default.patch1.45 KB

Added description text.

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Fixed

Committed and pushed. Thanks!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Minor modifications to specify that we are talking about the default option.

  • Commit 9846b92 on 7.x-3.x, 7.x-3.x-i18n, 8.x-3.x by merlinofchaos: 
    Issue #1836716 by grndlvl: Allow selection of default pipeline for new...