Support CCK multigroups
netaustin - September 28, 2009 - 19:03
| Project: | Tabs (jQuery UI tabs) |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Jump to:
Description
This patch enables support for CCK multigroups, which at present break cck_fieldgroup_tabs. The issue is that multigroups wrap themselves in a DIV for AHAH callbacks, which cause the tab DIV to be wrapped as well, and then lost when the user triggers multigroup's AHAH functionality, typically by clicking "add another value."
This does actually needs to be a patch to the tabs module, not cck_fieldgroup_tabs, because the relevant code is in the pre_render and theme functions.
I'm not totally comfortable with the CSS used here to format the fieldset properly after the AHAH call, but if you need multigroups and tabs *right now*, as I do, this patch will do it.
| Attachment | Size |
|---|---|
| tabs-with-multigroup.patch | 2.92 KB |
#1
Thanks for the patch.
I haven't used multigroups yet so I'm not clear on the details of what's needed.
I don't want tabs module to have logic specific to other modules, so what I think we need to do here is change the cck multigroup code to set #prefix and #suffix properties on the tab pages and, I guess, a [#attributes][class] property, and then run it through drupal_attributes() in the theme function.
#2
Subscribing.
Multigroups also seem to break vertical_tabs which like cck_fieldgroup_tabs is dependent on Tabs module. The issue crops up also when clicking "add more values".
#3
Subscribing
#4
subscribing