I just required to do for a custom development, and I've create to mini-modules with the relevant code showing how to integrate your own node settings form alteration with vertical tabs.

Probably it may require some English grammar and general review, but could be a start point for others to do so, and may end in a handbook page. The zip file contains just .module and .js required to alter the summary of the vertical tab created.

Comments

dave reid’s picture

Lets add these to Examples! That way we can easily point to them from the new handbook page!

dave reid’s picture

Status: Needs review » Needs work

And please make sure to update for the latest code.

ilo’s picture

Project: Vertical Tabs » Examples for Developers

Moving here. I'm confused Dave, what do you mean with latest?

ilo’s picture

Title: Examples of module integration for d6 and d7 » Examples of vertical tabs usage for D6 and D7
rfay’s picture

StatusFileSize
new4.67 KB

Committed the D6 version: http://drupal.org/cvs?commit=326886

rfay’s picture

Version: 6.x-1.x-dev »
Status: Needs work » Fixed
StatusFileSize
new5.11 KB

I'm committing the D7 version. Thanks for the work on this, Ilo!
http://drupal.org/cvs?commit=326896

This still has no tests.

dave reid’s picture

Assigned: Unassigned » dave reid
Status: Fixed » Active

There's really not much tests we can do for this since most of it happens in JavaScript. I'll see what if I can come up with something quick though. I do have to go through and fix a few things for the latest D6 vertical tabs module and show how to add vertical tabs on your own form.

dave reid’s picture

Status: Active » Fixed

Ok, added cleanups and more documentation. Thanks again ilo!

rfay’s picture

@davereid, please use the patch workflow when you're working. I've added a bit about this on the front page. Since we have the bot checking now, please attach your patch to the issue anad set to "needs review" as you would with a core patch. As a committer, you're free to commit after you get the green. I know there are no tests on this and no risk, but IMO that workflow will benefit us.

Thanks for your work on this!

dave reid’s picture

@rfay: Sorry about that. Will do from here on out!

Status: Fixed » Closed (fixed)

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