Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.When using multiple instances of CKEditor Tabber on one page, the left positioning of the DT's(tabs) isn't correct anymore. (Except for the first instance)
This happens because the tabIndex has been set as a global var to the plugin's scope and get's incremented everytime it sees a new DT(tab).
So the tabIndex get's only bigger while it should start at 0 again for every instance.
For e.g.: if I have 2 instances of the CKEditor Tabber, the tabIndex for the second instance might be starting already at 2 (instead of 0).
So the code thinks that the first tab of the 2nd CKEditor Tabber instance is the 3rd tab, while it should be seen as the first.
This breaks the calculation of the positioning of the tabs.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2774763_multiple_instances_of_ckeditor_fix.patch | 520 bytes | dylancompanjen |
Comments
Comment #2
dylancompanjen CreditAttribution: dylancompanjen commentedComment #3
dylancompanjen CreditAttribution: dylancompanjen commentedComment #4
dylancompanjen CreditAttribution: dylancompanjen at One Shoe commentedComment #5
dutchyodaThis patch works for me.
Comment #6
Tommy_001 CreditAttribution: Tommy_001 commentedI only have one instance of ckeditor tabber on my page, but the left positioning didn't work properly when logged in as admin until I applied your patch (it worked fine when logged out, strange enough. Thanks for this patch!
Comment #8
SebCorbin CreditAttribution: SebCorbin at Makina Corpus commentedPatch applied and release created, thanks!