Comments

andrewmacpherson’s picture

Status: Active » Needs review
StatusFileSize
new11.32 KB

Here's my patch...

If vertical_tabs.module is enabled, node_form_rearrange_form_alter() now checks to see if a vocabulary should be in a vertical tab. If so, it wraps the vocabulary field in a fieldset and attaches the supporting javascript file.

(I hope the patch works works, since it adds a new javascript file. I followed the cvsdo method for including new files in a patch.)

andrewmacpherson’s picture

Issue tags: +vertical tabs

adding 'vertical tabs' tag to issue

chiddicks’s picture

Has anyone else tried this patch?

giorgosk’s picture

Status: Needs review » Needs work
StatusFileSize
new2.97 KB
new6.35 KB

tried patch from #1 with latest dev
did not apply cleanly

I cleaned it A LOT
and applied it by hand

still not working
I don't see the newly created .js file inserted anywhere thus its not doing anything
hence still needs work

giorgosk’s picture

StatusFileSize
new1.86 KB

sorry my patch does not include the .js file (which as I said I don't know how it should be called)