-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
[Feature request] Show browser error overlay for runtime errors #2076
Comments
Sometimes runtime errors occur after most of the page is loaded normally and the page can be used normally. In this case, will the page be replaced with the browser error overlay? |
In React they prevent partial page from being rendered if there is an error, so error overlay is appropriate in all cases. It would be a nice enhancement particularly for new users and is present in CRA, NX, Snowpack etc. |
The overlay is an overlay so it's not replaced, it's overlaid or am I missing something here? This does the trick for me: // REGISTER ERROR OVERLAY
const showErrorOverlay = err => {
// must be within function call because that's when the element is defined for sure.
const ErrorOverlay = customElements.get('vite-error-overlay')
// don't open outside vite environment
if (!ErrorOverlay) {return}
console.log(err)
const overlay = new ErrorOverlay(err)
document.body.appendChild(overlay)
}
window.addEventListener('error', showErrorOverlay)
window.addEventListener('unhandledrejection', ({reason}) => showErrorOverlay(reason)) |
@FranzSkuffka Almost but the overlay needs the frame to be defined on the error to get correct formatting. The code you posted will essentially show Error: boom
at http://localhost:3000/src/index.tsx:17:7 vs what I think most people expect Runtime error at [filepath]
[error message]
/[filepath]:17:0
17 | throw new Error('boom')
18 | ^
at [stacktrace] On an unrelated note small bug in the code you posted, I think you meant to pass in the error not the event. window.addEventListener('error', ({error}) => showErrorOverlay(error)); |
Please prioritise this one, we are all spoilt by our previous experiences with webpack and other dev-servers. Really hard to live without the run-time overlay. Thanks! |
Thanks @arbassett for the related PR (#6274) you are working on. Will this PR cover showing all runtime errors in the overlay? Would be great if it does. Here is example where prop validations fail (validator fn & type check). At the moment, we are forced to keep the browser console always open to even notice these important errors. |
I thought i was doing something wrong, but it really is a problem. I would really love to see this improved |
Fantastic, seems to have solved it for now. |
I'm able to make the overlay more usable with information such as stacktrace using if (import.meta.env.DEV) {
window.onerror = (event, source, lineno, colno, err) => {
// must be within function call because that's when the element is defined for sure.
const ErrorOverlay = customElements.get('vite-error-overlay');
// don't open outside vite environment
if (!ErrorOverlay) {
return;
}
const overlay = new ErrorOverlay(err);
document.body.appendChild(overlay);
};
} Example using
|
Where do you precisely add this code @vKongv for it to work? I've tried adding it in the |
@c10b10 you can dump it into |
Not sure if there's a better way to do this at this point, but for ErrorEvents, you also need to pass the window.addEventListener('error', ({error}) => showErrorOverlay(error)); // here |
A little hint for those who are using typescript and would like to switch the prop import type { ErrorPayload } from 'vite'
export const showErrorOverlay = (err: Partial<ErrorPayload['err']>) => {
const ErrorOverlay = customElements.get('vite-error-overlay')
if (ErrorOverlay == null) return
document.body.appendChild(new ErrorOverlay(err))
} |
This comment has been minimized.
This comment has been minimized.
Any solution for stack trace showing different lines than actual error position? |
I did some work on this using a vite plugin instead. |
I really like the browser error overlay that we get for compile errors (server.hmr.overlay). But runtime errors still appear only in console. It would be nice if runtime errors could also be displayed using the nice browser error overlay.
For example, it would be nice to get the overlay if I forgot to import ref:
I'm talking about this overlay:
The text was updated successfully, but these errors were encountered: