Table tr are showing wrong classes. See attachment:
error

CommentFileSizeAuthor
#6 1907050-6--row_zebra_redux.patch1.31 KBdrunken monkey
#5 1907050-row-zebra-redux.patch2.9 KBmradcliffe
#1 0001-Issue-1907050-by-mradcliffe-TonyxTony-Fix-row-zebra-.patch2.14 KBmradcliffe
TEST.png8.6 KBTonyxTony
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Status: Active » Needs review
FileSize
0001-Issue-1907050-by-mradcliffe-TonyxTony-Fix-row-zebra-.patch2.14 KB

Zebra classes are being set before sorting.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Issue summary: View changes
Status: Needs review » Fixed

Committed after a year of NR: b0290ba0a469cb7ab7e09f3f640741f2f4255007

Status: Fixed » Closed (fixed)

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

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Status: Closed (fixed) » Needs work

Tested locally and I see incorrect classes again.

mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Status: Needs work » Needs review
FileSize
1907050-row-zebra-redux.patch2.9 KB

Take two.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey commented
FileSize
1907050-6--row_zebra_redux.patch1.31 KB

Here is a re-roll for the latest dev version.

herd45’s picture

herd45 CreditAttribution: herd45 commented

I've tested the patch in #6 and it works as expected

  • mradcliffe committed cdf4cb0 on 7.x-1.x 
    Issue #1907050 by mradcliffe, drunken monkey, herd45: Fix zebra striping...
mradcliffe’s picture

mradcliffe
he/him
Primary language English
CreditAttribution: mradcliffe commented
Status: Needs review » Fixed

Tested and pushed.

Status: Fixed » Closed (fixed)

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