Play nicely with Panels [#465824]

Setting a panels 3 edit node 'task handler' would over write any settings in this module. It would be super good to get it to play nicely together.

The ideal implementation would have nfc alter the 'general form' panel of panels, and allow panels to set that and other panels around it.

Comment	File	Size	Author
#5	465824-nodeformcols.diff	727 bytes	wulff
#5	465824.tar_.gz	2.1 KB	wulff

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Hugo Wetterberg CreditAttribution: Hugo Wetterberg commented 18 May 2009 at 19:46

Assigned:

Unassigned

» Hugo Wetterberg

I haven't done any work with panels since I was too green to know how to code a theme :)
But it's a great module, and it stands to reason that I should support it if it's possible.
I'll try to remember to get back to you if I don't find an easy fix.

Comment #2

Hugo Wetterberg CreditAttribution: Hugo Wetterberg commented 18 May 2009 at 20:41

Status:

Active

» Postponed

Hmm, it looks like there's no easy fix to this. I rearrange the form, and so does panels, to negotiate a agreement between the modules is probably pretty hard.

The one solution that springs to mind is that the nfc-form could be added as panel contents. But I couldn't even get panels to work correctly for node forms without nfc activated, so I think that I'll wait until panels is out in a more final form.

If you, or anyone else, have any thoughts or patches, they'll be very welcome. But for now this issue will be postponed.

Comment #3

Flying Drupalist CreditAttribution: Flying Drupalist commented 20 May 2009 at 11:56

Thanks for looking at this. Well I tried to use panels on node forms with nfc and it just seems like panels is overriding nfc. So for me panels is working fine. Perhaps what nfc can do is, when a panel is detected, rather than overwrite the form, find the 'general forms' panel in panels and overwrite that instead.

Comment #4

Hugo Wetterberg CreditAttribution: Hugo Wetterberg commented 21 May 2009 at 18:56

Yeah. I would appreciate any help in this matter, as I'm not very familiar with panels. Any kind of prototype code/patch/pointer on how to get that working would be appreciated.

/Hugo