Not sure, if this even is needed. In nodequeue module the colspan for the table listing queues is set to 7, can we calculate it from count of values in $header. Attaching small patch doing the same.

CommentFileSizeAuthor
#4 nodequeue-change_colspan_to_be_dynamic-2292381-4.patch760 bytesjenlampton
#3 nodequeue-change_colspan_to_be_dynamic-2292381-3.patch492 bytesjenlampton
#1 changing-colspan-to-be-dynamic-2292381-1.patch856 bytesarpitr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arpitr’s picture

arpitr CreditAttribution: arpitr commented
FileSize
changing-colspan-to-be-dynamic-2292381-1.patch856 bytes
arpitr’s picture

arpitr CreditAttribution: arpitr commented
Assigned: arpitr » Unassigned
Status: Active » Needs review
jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton as a volunteer and at Jeneration Web Development for BGP Site Solutions commented
Version: 7.x-3.x-dev » 7.x-2.x-dev
FileSize
nodequeue-change_colspan_to_be_dynamic-2292381-3.patch492 bytes

rerolled.

jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton as a volunteer and at Jeneration Web Development for BGP Site Solutions commented
FileSize
nodequeue-change_colspan_to_be_dynamic-2292381-4.patch760 bytes

whoopsie, let's try this one instead.

Status: Needs review » Needs work

The last submitted patch, 4: nodequeue-change_colspan_to_be_dynamic-2292381-4.patch, failed testing.

The last submitted patch, 4: nodequeue-change_colspan_to_be_dynamic-2292381-4.patch, failed testing.

The last submitted patch, 4: nodequeue-change_colspan_to_be_dynamic-2292381-4.patch, failed testing.