-
Notifications
You must be signed in to change notification settings - Fork 9.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Runtime Error Design Update #9820
Comments
thicc bang |
Here is a zip for the error icons in svg: ic_errors.zip |
@yuinchien I'd like to work on this issue. Just to confirm, its simply to update all current error icons with new design in lighthouse performance report, Right? |
Thanks @rahulpatel596! Basically. We want to replace the red triangle with the new red exclamation icon for all audits that error. Sidenote for Lighthouse team, what do you think about changing the text from "Error" to "Exception Thrown"? I feel like that might clear up most of the confusion around whether this is their failure or ours. |
Hey @patrickhulce |
We don't want to replace the legend one, other than that yes those 3 places would have the exclamation instead of the question mark/triangle and the regular audits below this as well. Keep in mind that this exclamation design should only apply when there has been an error and the score is |
Hey @patrickhulce Is there a specific method or way to follow which can generate report with an error and/or with null score? |
Yes @rahulpatel596, every PR has a sample error report deployed and you can replicate it on your local machine with |
Some good progress was made in #11670 but this is available again if anyone would like to pick it up. |
I'm happy to work on this, but it seems there is already a PR (#9272) that handles updating the error icons. |
@tannerdolby that PR is very old, you are welcome to submit your own :) |
@adamraine sounds good, I will start working on this and submit a PR soon :) |
The text was updated successfully, but these errors were encountered: