Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#30981 closed defect (bug) (fixed)

When editing the tags in the table, the table's alternate rows don't udate.

Reported by: kevdotbadger's profile kevdotbadger Owned by: helen's profile helen
Milestone: 4.2 Priority: normal
Severity: normal Version:
Component: Taxonomy Keywords: has-patch dev-feedback
Focuses: ui, javascript, administration Cc:

Description

Adding a row, or removing a row doesn't update the odd "alternate" rows.

Attachments (2)

30981.diff (1.1 KB) - added by kevdotbadger 9 years ago.
Fixes 30981
30981.2.diff (993 bytes) - added by kevdotbadger 9 years ago.

Download all attachments as: .zip

Change History (9)

@kevdotbadger
9 years ago

Fixes 30981

@kevdotbadger
9 years ago

#1 @kevdotbadger
9 years ago

  • Keywords has-patch dev-feedback added

#2 @kevdotbadger
9 years ago

  • Focuses javascript added

#3 @SergeyBiryukov
9 years ago

  • Component changed from Administration to Taxonomy

#4 @helen
9 years ago

I think I am going to try to wipe out all of these problems in #26060.

#5 @kevdotbadger
9 years ago

Sounds like a good idea, i wondered why we were a JS solution, rather than CSS. I assumed it was for older browsers.

#6 @helen
9 years ago

  • Milestone changed from Awaiting Review to 4.2

#7 @helen
9 years ago

  • Owner set to helen
  • Resolution set to fixed
  • Status changed from new to closed

In 31181:

List tables: Use CSS :nth-child() selectors for zebra striping.

Note that this does not fix issues related to comment quick edit. Internal linking also continues to use the .alternate class for now. IE8 and below gracefully degrade by not having zebra striping.

There is some hoop jumping with adding an extra table row to maintain zebra striping during quick edit. Documenting that here for future reference; it is also in the inline documentation.

fixes #30981 and #26060. see #25060.

Note: See TracTickets for help on using tickets.