I recognised that the #default_value of vertical_tabs was not working and digged in.

They changed somewhat the naming synopsis .... somehow it still seems more or less to work.

The patch i provide here only fixes the default value. I guess the complete file (vertical-tabs.js) need to be merged with core changes.

CommentFileSizeAuthor
#3 vertical_tabs_merge-2830195-1.patch599 bytesytsurk
#2 vertical_tabs_merge-2830195-1.patch599 bytesytsurk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ytsurk created an issue. See original summary.

ytsurk’s picture

ytsurk
Location Switzerland
CreditAttribution: ytsurk commented
FileSize
vertical_tabs_merge-2830195-1.patch599 bytes

Here's the patch to make the default value (inital selected tab) work again

ytsurk’s picture

ytsurk
Location Switzerland
CreditAttribution: ytsurk commented
Issue summary: View changes
FileSize
vertical_tabs_merge-2830195-1.patch599 bytes

this is always the same file ..

  • markcarver committed d4d54a8 on 8.x-3.x authored by ytsurk 
    Issue #2830195 by ytsurk: Vertical Tabs js was updated in core and needs...
markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented
Status: Active » Fixed
ytsurk’s picture

ytsurk
Location Switzerland
CreditAttribution: ytsurk commented

So you don't care about the other changes they made ;) ?

markhalliwell’s picture

markhalliwell
Primary language English
CreditAttribution: markhalliwell at Tag1 Consulting commented

Meh.

If it's broken, sure. I'm not entirely sure what all has changed since the original port TBH.

If you feel like doing another patch, then sure, please feel free to open this issue back up and I'll apply that too.

Status: Fixed » Closed (fixed)

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