Closed Bug 914831 Opened 11 years ago Closed 9 years ago

/daily Graph and legend colors should match the colors chosen for the table headers in the Crashes per ADI table.

Categories

(Socorro :: Webapp, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mbrandt, Assigned: thalia.e.chan)

References

()

Details

Attachments

(2 files)

Graph colors/legend colors do not match product version colors in Crashes per ADI table. Note this is not a regression. Steps to reproduce: 1. goto /daily and select several product versions https://crash-stats.allizom.org/daily?form_selection=by_version&p=Firefox&v=23.0.1&v=24.0b&v=24.0b10&v=24.0b1&hang_type=any&os=Windows&os=Mac+OS+X&os=Linux&date_range_type=report&date_start=2013-08-27&date_end=2013-09-10&submit=Generate https://crash-stats.mozilla.com/daily?form_selection=by_version&p=Firefox&v=24.0b1&v=26.0a1&v=24.0b&v=23.0.1&hang_type=any&os=Windows&os=Mac+OS+X&os=Linux&date_range_type=report&date_start=2013-08-27&date_end=2013-09-10&submit=Generate Actual: Graph colors/legend colors do not match product version colors in Crashes per ADI table. Expected: Graph and legend colors should match the colors chosen for the table headers in the Crashes per ADI table.
Additional screenshot of the bug on prod
isn't this a regression? Or perhaps I've not been paying attention to graph labels + colors.
(In reply to Wayne Mery (:wsmwk) from comment #2) > isn't this a regression? Or perhaps I've not been paying attention to graph > labels + colors. It may be a regression but I don't have the history behind this feature to know whether it is one or not.
Brandon has been working on this code recently.
Assignee: nobody → thalia.e.chan
Status: NEW → ASSIGNED
This happens because the graph assigns colours to the versions once they've been sorted alphabetically, whereas the table does not. (The above images are from the old graph library but the same happens with the new graph library.) A related problem is that when the same version is chosen twice in the form, the graph intelligently draws 3 lines and assigns 3 colours (instead of 4), but the table does not: it shows two copies of the repeated version and assigns each copy a different colour, so inevitably the colours in the graph and the table don't match. I suggest that the versions be sorted and made unique in the view. This will make the colours match and will prevent the table from having repeated columns.
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/f6b5633e95f6a5c015b2840a1082b1a604c8a2e7 Fixes Bug 914831 - /daily Graph and legend colors should match the colors chosen for the table headers https://github.com/mozilla/socorro/commit/23d96e1b68eb5abfd1ed5faaf8932fd6ce8f92a9 Merge pull request #2919 from Tchanders/crashes-adi-versions Fixes Bug 914831 - /daily Graph and legend colors should match the co…
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: