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.

CommentFileSizeAuthor
#5 465824-nodeformcols.diff727 byteswulff
#5 465824.tar_.gz2.1 KBwulff

Comments

Hugo Wetterberg’s picture

Hugo Wetterberg commented
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.

Hugo Wetterberg’s picture

Hugo Wetterberg commented
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.

Flying Drupalist’s picture

Flying Drupalist commented

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.

Hugo Wetterberg’s picture

Hugo Wetterberg commented

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

wulff’s picture

wulff commented
Status: Postponed » Active
StatusFileSize
new 465824.tar_.gz2.1 KB
new 465824-nodeformcols.diff727 bytes

One prototype coming up :-)

This lets you add the complete nodeformcols form as a pane.

Hugo Wetterberg’s picture

Hugo Wetterberg commented
Version: 6.x-1.x-dev » 6.x-1.6
Status: Active » Fixed

Thanks Morten, works beautifully, I've added it to the 6.x-1.6 release of nodeformcols.

Cheers,
Hugo

Status: Fixed » Closed (fixed)

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