Create-react-app: react-error-overlay slow?

Created on 21 Dec 2018  ยท  5Comments  ยท  Source: facebook/create-react-app

I was wondering if it is a known issue that react-error-overlay is very slow, I'm currently using a package that e.g. calls console.error a lot, and the fact that I'm using create-react-app means that the native console.error is overwritten (affecting this particular package).

So my main question: is this an issue with the react-error-overlay implementation of console.error being slow or should it be expected that it can't handle thousands of repeated calls compare to native console.error?

I'm surprised that console.error is hijacked, I didn't expect that (when I was looking into the issue).

Most helpful comment

Hey, if no one is working on it I'd like to give a try to it. I'm basically in the same situation as @mohsinulhaq (original creator of #6098) โ€“ i'm using error-overlay-webpack-plugin to get react-error-overlay into my application. Since the app is pretty big (๐Ÿ˜ž) the downloading/parsing of source map takes significant time and since it's blocking rendering of the actual error I would like to tackle it.

I agree it would make sense to show compiled stacktrace immediately until the sourcemaps are loaded.

What do you think?

All 5 comments

It is pretty slow right now, we're looking at making it faster. ๐Ÿ˜„

Has this been taken up yet? It would be great if compiled stacktrace is shown immediately after error until sourcemapped stacktrace is computed. Just like redbox-react.
Thanks.

No, if you'd like to contribute you can send a PR!

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 5 days if no further activity occurs.

Hey, if no one is working on it I'd like to give a try to it. I'm basically in the same situation as @mohsinulhaq (original creator of #6098) โ€“ i'm using error-overlay-webpack-plugin to get react-error-overlay into my application. Since the app is pretty big (๐Ÿ˜ž) the downloading/parsing of source map takes significant time and since it's blocking rendering of the actual error I would like to tackle it.

I agree it would make sense to show compiled stacktrace immediately until the sourcemaps are loaded.

What do you think?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

AlexeyRyashencev picture AlexeyRyashencev  ยท  3Comments

fson picture fson  ยท  3Comments

ap13p picture ap13p  ยท  3Comments

rdamian3 picture rdamian3  ยท  3Comments

oltsa picture oltsa  ยท  3Comments