There is no reason that panel_nodes shouldn't have at least some limited context support. This issue is to track progress towards adding such support.

CommentFileSizeAuthor
#8 panels-DRUPAL-5--2.node-context.patch6.97 KBsun
#7 panels-DRUPAL-5--2.node-context.patch7.49 KBsun
#5 panels-DRUPAL-5--2.node-context.patch6.31 KBsun

Comments

ugerhard’s picture

ugerhard commented
Title: Add content to panel nodes » Add context to panel nodes

Just changing the title, as I assume "Add context to panel nodes" was meant, not "Add content to panel nodes" :-)

sdboyer’s picture

sdboyer commented

HAH. Clearly I should have been sleeping, not transferring my to-do list into the issue queue. Thanks!

najibx’s picture

najibx commented

not sure if my comment should be here ..but og_panel, and panel_page doesnt not support context, if im not mistaken.

sdboyer’s picture

sdboyer commented

og_panels is part of og, and neither merlin nor I have commit access to it.

I assume you wrote 'panels_page' in error, as it's the clearest example of "supporting context."

More substantively, though - they DO 'support' context, they just don't feature a UI that allows you to configure it. Nothing wrong with that, if it's appropriate for the use case. The thrust of this issue is that there are some specific aspects of UI-based context configuration that should be added to panels_node.

It's my mistake for phrasing it in such a misleading way in the original issue, apologies.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Needs review
StatusFileSize
new panels-DRUPAL-5--2.node-context.patch6.31 KB

Whoooza.

Attached patch builds the ground-work for contexts in panel nodes. Not much, mostly copied'n'pasted from panels_page and adjusted for panels_node.

sdboyer’s picture

sdboyer commented

AWESOME. Thanks so much for laying the groundwork here, sun - I wouldn't have gotten to this anytime soon, that's for sure. I'll have a look this weekend during my Panels work time.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Assigned: sdboyer » sun
Category: task » feature
StatusFileSize
new panels-DRUPAL-5--2.node-context.patch7.49 KB

Yay! Now working properly =)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
StatusFileSize
new panels-DRUPAL-5--2.node-context.patch6.97 KB

Now without argument handling, because arguments will not be supported in panel nodes.

sdboyer’s picture

sdboyer commented
Assigned: sun » sdboyer

Assigning to me so that I'll actually get to this soon...

merlinofchaos’s picture

merlinofchaos commented
Status: Needs review » Patch (to be ported)

Looks good. Setting status to forward port this to the D6 version.

esmerel’s picture

esmerel commented
Status: Patch (to be ported) » Closed (won't fix)

No features are going to be added to the 2.x line, and there's a bunch of context related feature requests in the 3.x line.