Skip to content
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

Closed
yuinchien opened this issue Oct 11, 2019 · 13 comments · Fixed by #15092
Closed

Runtime Error Design Update #9820

yuinchien opened this issue Oct 11, 2019 · 13 comments · Fixed by #15092

Comments

@yuinchien
Copy link
Collaborator

75

76

@exterkamp
Copy link
Member

Note for when we fix this: Fix the i18n of Error for the metrics, and maybe pull the same string?
image

@paulirish
Copy link
Member

thicc bang

@yuinchien
Copy link
Collaborator Author

76

Here is a zip for the error icons in svg: ic_errors.zip

@rahulpatel596
Copy link

rahulpatel596 commented Nov 4, 2020

@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?
Not sure about second error with i18n error, but might just get started first.. 😄

@patrickhulce
Copy link
Collaborator

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.

@rahulpatel596
Copy link

rahulpatel596 commented Nov 8, 2020

Hey @patrickhulce
I am working on this and I just wanted to confirm that I am making changes in correct place because current production version has a quite different UI.
Thanks for your help :)

Screen Shot 2020-11-07 at 9 26 12 PM

@patrickhulce
Copy link
Collaborator

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 null.

@rahulpatel596
Copy link

Hey @patrickhulce Is there a specific method or way to follow which can generate report with an error and/or with null score?

@patrickhulce
Copy link
Collaborator

Yes @rahulpatel596, every PR has a sample error report deployed and you can replicate it on your local machine with yarn now-build

@patrickhulce
Copy link
Collaborator

Some good progress was made in #11670 but this is available again if anyone would like to pick it up.

@tannerdolby
Copy link
Contributor

tannerdolby commented May 11, 2023

I'm happy to work on this, but it seems there is already a PR (#9272) that handles updating the error icons.

@adamraine
Copy link
Member

@tannerdolby that PR is very old, you are welcome to submit your own :)

@tannerdolby
Copy link
Contributor

@adamraine sounds good, I will start working on this and submit a PR soon :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants