Backport to Drupal 5.x

I have made quite a few changes, so please update.

I would prefer that you copy Drupal.t (with modifications), Drupal.checkPlain, and Drupal.formatPlural (with modifications) from Drupal 6, so the difference between the two versions becomes smaller.

Considering the amount of work needed to get this in drupal seven I think efforts are not well spent backporting this. nodeform module is stable and provides the most important functionality of this module (vertical tabs on node forms) and would likely be easier to backport.

Having said that, this is open source, so scratch your own itch! :)

This isn't working for me, tested on 2 sites..

JQuery Interface Library 5.x-1.0
jQuery Update 5.x-2.0

^ are both installed and enabled on at least one of the sites I tested it on

Would my theme make any difference? I'm using Marinelli

It's working now, but the display css still needs some work.. that could be related to my theme though.. about to test on another site

Edit: After testing on another site display appears to be variable based on template but some additions to the CSS could still be useful..

Here is my current version..

.vertical-tabs-list .description {
  display: block;
}

/* Border style of tabs and content */
div.vertical-tabs-div,
ul.vertical-tabs-list {
  border: 1px solid #D9EAF5;
  list-style:none;
  padding-left:0;
}

/* Make space for the tabs on the side */
.vertical-tabs {
  margin-left: 180px;
}

/* Set the background image on the tabs and content */
.vertical-tabs,
.vertical-tabs ul.vertical-tabs-list {
  background: #fff url(gradient-inner.png) repeat-x top;
}

/* Position and layout of tabs container */
.vertical-tabs ul.vertical-tabs-list { 
  width: 179px;
  float: left;
  margin: 0 0 0 -181px;
  border-right: 0px;
  z-index: 1;
  border-bottom: 0;
}


/* Reset LIs which probably have awkward layout inherited from tabs */
.vertical-tabs ul.vertical-tabs-list li {
  float: none;
  margin: 0;
  padding: 0;
  background-image: none;
}

/* Layout of each tab */
.vertical-tabs ul.vertical-tabs-list a {
  display: block;
  margin: 0;
  padding: .4em .3em .1em .6em;
  background: #EDF5FA;
  text-align: left;
  font-weight: normal;
  border-bottom: 1px solid #D9EAF5;
}

.vertical-tabs ul.vertical-tabs-list a.vertical-tabs-nodescription {
  padding-bottom: .4em;
}

.vertical-tabs ul.vertical-tabs-list .description {
  padding: 0;
  line-height: normal;
  min-height: 0;
  white-space: normal;
}

.vertical-tabs ul.vertical-tabs-list a {
  border-right: 1px solid #D9EAF5;
  text-decoration: none;
}

/* Hover state of tabs */
.vertical-tabs ul.vertical-tabs-list a:hover {
  color: #0062A0;
  text-decoration: none;
  cursor: pointer;
}

.vertical-tabs ul.vertical-tabs-list a:focus {
  outline: none;
  text-decoration: none;
}

/* Selected tab */
.vertical-tabs ul.vertical-tabs-list li.selected a {
  background: transparent none;
  color: black;
  border-right: 0;
}

/* Reset some important elements of each panel */
.vertical-tabs .vertical-tabs-div {
  margin: 0;
  padding: .5em;
  margin-top: -1px;
  margin-left: -1px;
  background: transparent none;
}

.buttons {
  clear: both;
}

#edit-book-bid-wrapper .description {
  clear:none;
}

.vertical-tabs .description {
color:#898989;
font-size:0.9em;
}

subscribe

subscribe

I've been trying to use the patches with the current version of vertical tabs (6.x-1.x-dev), but the sections of the original to be replaced don't exist. Has vertical tabs been updated so that the patches are no longer appropriate for the current version?

Thanks.

Could you make available all the modified files that you're using? The current release of Vertical Tabs has been updated since the patches were written, so the patches can't be applied to the newer files.

Thanks
RWW

I'm fine with providing a backport if these patches are updated. Try to keep changes minimal to provide ease of maintenance.

subscribing

subscribing

subscribing

subscribing

subscribing

subscribing, and I think this should go back to normal priority - I would so much like this in my 5.x site!

If you really want to get this backported, make sure to keep the patches up to date.

Attached is a new patch to get things rolling again, no way near committable yet, but it does somewhat work.

Cheers,
Deciphered.

Updated patch.

Should now be good for review, fixed all issues I was aware of from the last patch.
Only bugs that I have noticed are also visible in D6 versions.

TODO: Fix support for Garland/Color module.

Hey Deciphered,

Thanks so much for updating this. I patched the latest 6.1-dev version, installed it on a latest D5 site, and get the following whenever visiting a node-type page:

    * warning: include_once(modules/node/node.pages.inc) [function.include-once]: failed to open stream: No such file or directory in /var/www/thomas/lc3/sites/all/modules/vertical_tabs/vertical_tabs.module on line 111.
    * warning: include_once() [function.include]: Failed opening 'modules/node/node.pages.inc' for inclusion (include_path='.:/usr/share/php:/usr/share/pear:/var/www/thomas/lc3/sites/all/modules/activitystream/activitystream_flickr/PEAR') in /var/www/thomas/lc3/sites/all/modules/vertical_tabs/vertical_tabs.module on line 111.
    * warning: uasort() [function.uasort]: Invalid comparison function. in /var/www/thomas/lc3/sites/all/modules/vertical_tabs/vertical_tabs.module on line 127.

Seems to work fine regardless of this message.

@pribeh

Thanks for the feedback. I really should have the development build of Drupal 5 running so I get those errors as well, will install it shortly.
The issue has, or should have, been fixed in this latest patch.

Also in this latest patch is some initial work towards the garland/color stuff, but still needs work.

Cheers,
Deciphered.

Expect updated patch within the next couple of days.

Thanks Deciphered, after doing the D7 backport of CSS I think there'll be less overall changes needed before a final release. It'll be great to have another review of the latest CVS code before making a new beta. Thanks!

Hi, I applied the latest patch but all I get at the bottom page is "[object Object]" in a fieldset
Thanks

No activity for almost six months means this is a won't fix.