Steps to reproduce:
- load page with sticky table header
- scroll down to see sticky tableheader
- run js code Drupal.attachBehaviors(<some html element>);

Now sticky tableheader is visible always and does not disappear on scroll

CommentFileSizeAuthor
#4 1236636_tableheader.js_.patch1.15 KBvectoroc
#1 1236636_tableheader.js_.patch1.05 KBvectoroc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vectoroc’s picture

vectoroc CreditAttribution: vectoroc commented
FileSize
1236636_tableheader.js_.patch1.05 KB

I noticed this happens because Drupal.behaviors.tableHeader overwrites Drupal.tableHeaderOnScroll function with headers variable

patch makes headers variable incremental

vectoroc’s picture

vectoroc CreditAttribution: vectoroc commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1236636_tableheader.js_.patch, failed testing.

vectoroc’s picture

vectoroc CreditAttribution: vectoroc commented
Status: Needs work » Needs review
FileSize
1236636_tableheader.js_.patch1.15 KB
vectoroc’s picture

vectoroc CreditAttribution: vectoroc commented
Title: sticky tableheader are broken after Drupal.attachBehaviors » AHAH breaks sticky tableheaders
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Version: 6.x-dev » 8.x-dev
Status: Needs review » Needs work
Issue tags: +Needs backport to D6, +Needs backport to D7

It seems 7.x affected too, so we need to fix this in current (8.x) first then backport

+++ b/misc/tableheader.jsundefined
@@ -86,15 +90,15 @@ Drupal.behaviors.tableHeader = function (context) {
+  ¶

trailing whitespace

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Status: Needs work » Closed (cannot reproduce)

Doesn't happen on 8.x

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ commented
Issue summary: View changes

wrapped code with tag

vectoroc’s picture

vectoroc CreditAttribution: vectoroc commented
Version: 8.x-dev » 7.x-dev
Issue summary: View changes
Status: Closed (cannot reproduce) » Patch (to be ported)