Hi,

First I want to say great module. Now for the feature request ;-)

It'll be nice if you can create vertical tabs with the click of a checkbox. We needed this for a project so here's a quick patch to make that work. Preferably in the long run it'll be nice to support multiple styles so we can have something like "vertical tabs", "tabs accross the top", "tab in the right side", ..... and the list goes on.

Sorry for not being able to patch agains cvs :-/

I also needed to add a css file to display the vertical tabs vertically.

Attached is the patch and the css file. The patch file just add's a 'Use Vertical tabs' checkbox to the views options, then adds and extra class to the tabset, and it also add the css.

The css just has 2 rules to overwrite the tabs css.

All comments welcome.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ericduran’s picture

Actually I didn't realized this module was supporting fields and node row style so I had to do some changes.

Old

      $form['vertical_field'] = array(
        '#title' => t('Use vertical tabs'),
        '#type'  => 'checkbox',
        '#default_value' => $options['vertical_field'],
      );

New

    $form['vertical_field'] = array(
      '#title' => t('Use vertical tabs'),
      '#type'  => 'checkbox',
      '#default_value' => $this->options['vertical_field'],
    );

I should provide an appropriate patch in the next couple of days.

jdwfly’s picture

Status: Active » Needs work

I was able to use just CSS to provide vertical tabs on a page. Although it was easy enough to do it may be beneficial to have some options that allow for some default styling. This would need some more work and thought though before being viable to put into the module.

nedjo’s picture

Thanks for the patch.

I'd like to see this as a patch on tabs.module rather than views_tabs and also to be based on the CSS/JS of jquery_ui tabs as per the existing examples that ship with the current version.

See http://jquery-ui.googlecode.com/svn/trunk/demos/tabs/vertical.html.

ericduran’s picture

Yeah it makes more sense for the css to be on tabs rather then here. At 1st I had patch both modules but then decided to keep it all in one module.

Also it actually look like we're going to have to go a different route instead of using views_tabs for the project. The problem is that the views_tab module and jquery_ui module don't play nice together. We're using jquery_ui with 1.7 and the tabs module includes 1.6 which breaks everything.

What do you think of replacing the tabs module dependency with jquery_ui ? We'll actually that doesn't make sense because then you loose the tabset and tabpage theme functions. But how about integrating tabs with jquery ui?

I guess I'll move this to a separate issue in tabs.

Thanks.

jdwfly’s picture

Project: Views Tabs » Tabs (jQuery UI tabs)

Moving to Tabs since maintainer wants to see this go into that module.

kenorb’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)

Closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.