Report: add legend to decipher iconography left to us by the ancient ones #1841
Conversation
ebidel
changed the title
| @@ -120,7 +126,6 @@ summary { | |||
| margin: 0 auto; | |||
| max-width: var(--report-width); | |||
| background-color: #fff; | |||
| box-shadow: 0 0 6px 0 rgba(0,0,0,0.26); | |||
There was a problem hiding this comment.
this is no longer visible, so removing.
|
My my this is very attractive. Nice stuff, man. |
| </li> | ||
| <li> | ||
| <span class="aggregation__header__score score-poor-bg subitem-result__poor">Poor</span> | ||
| <span class="footer__key-description">Overall category score is < 73</span> |
There was a problem hiding this comment.
definitely clears things up. The current report is...hard to decipher.
I'd normally argue the fact that we need a legend for report scoring is a UX smell and an argument that we need to fix the report itself, but that's in progress so hopefully this isn't a permanent solution :)
100% agree. It's gotten out of control. I wanted to get this in before the report redo. No telling how long or what form that will take. |
* Report: add legend to decipher iconography left to us by the ancient ones (#1841) * Report: print doesn't cut off expanded audit details (#1870) * Report: increase icon size for a11y (#1856) * Tweak report colors so that we are WCAG2AA valid. * Biggin icons * CLI: add update-notifier. Fixes #1805 (#1890) * Fixes #1907 - move update-notifier to root (#1908) * CLI: remove npm prepublish (#1889)
R: all
Time for a legend kids! As we've added more icons, it's become unclear what everything means.