Hi,

This may or may not be a duplicate of #331772: active tabs, but the issue I have is that the tabs are not assigned the correct html and classes to render correctly using the default Drupal tabs style.

The UL needs class="tabs clear-block" and the LI needs a have a <span> inside the and wrapped around the title.

Fix in patch is attached.

CommentFileSizeAuthor
tabs-6.x-1.0-render_tabs_correctly-1.patch1021 bytesdeciphered

Comments

sunshinee’s picture

sunshinee commented

I wouldn't consider myself qualified to review the code, but I did apply the patch and it works beautifully. Thanks, Deciphered.

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs review » Reviewed & tested by the community

@rjoy

Glad it worked for you, and that's all that really counts.

As I've been using the patch since I wrote it without any issues I'm marking this as RTBC.

squarecandy’s picture

squarecandy commented

Works great for me. Thank you!!!

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks, applied.

Status: Fixed » Closed (fixed)

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