Search result table rows all colored the same

VERIFIED FIXED in 2.4.3

Status

Socorro
Webapp
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: lonnen, Assigned: espressive)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [regression])

(Reporter)

Description

6 years ago
Odd rows are colored by:
table.tablesorter tr:nth-child(2n+1) -> rgb(237, 236, 236)

Even are colored by: 
table.tablesorter tbody tr.odd -> rgb(237, 236, 236)

The latter is overriding the flora.tablesorters rule:
table.tablesorter tbody tr -> rgb(255,255,255)

Which causes uniform row coloring in the search results view. See https://crash-stats-dev.allizom.org/query/query?product=Firefox&version=Firefox%3A10.0.1&range_value=3&range_unit=weeks&date=02%2F14%2F2012+18%3A42%3A30&query_search=signature&query_type=contains&query=&reason=&build_id=&process_type=any&hang_type=any&do_query=1
(Assignee)

Comment 1

6 years ago
Sent pull request: https://github.com/mozilla/socorro/pull/349

TEST CASE
---------

Everywhere on Socorro where alternate table row colors are implemented the scheme should always be consistent with all other implementation elsewhere on the site.
Status: NEW → ASSIGNED
Target Milestone: --- → 2.4.3

Comment 2

6 years ago
Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/748118f8aa1072fb33d2d2fd22b1c3859b781874
Merge pull request #349 from ossreleasefeed/ensure-alternate-row-consistency-727162

alternate row colors now consistent across socorro fixes bug 727162

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(In reply to Schalk Neethling from comment #1)
> Sent pull request: https://github.com/mozilla/socorro/pull/349
> Everywhere on Socorro where alternate table row colors 

That's a tall order ;-) ... I've checked throughout the app on stage and the alternating row colors are back. Marking QA verified on stage.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.