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

react-error-overlay dark theme #6760

Closed
William-Owen opened this issue Apr 4, 2019 · 12 comments · Fixed by #7052
Closed

react-error-overlay dark theme #6760

William-Owen opened this issue Apr 4, 2019 · 12 comments · Fixed by #7052

Comments

@William-Owen
Copy link

As the tech is increasingly moving over to dark themes; is it possible to get the react-error-overlay to show runtime errors in a dark theme, or possibly making it an option?

Working at night on an application that has a dark theme on my dark themed IDE running on my dark theme OS I find when I cause a runtime error not only am I sad because of the runtime error but also I am sad as I am now blind; which makes solving the error more challenging.

@iansu
Copy link
Contributor

iansu commented Apr 4, 2019

We're not necessarily opposed to this idea, as long as it's unobtrusive (for example by using the prefers-color-scheme: dark media query). However, the only way this is going to happen is if someone from the community does the work and sends in a PR. Is this something you want to work on?

@kostia-lev
Copy link

kostia-lev commented Apr 5, 2019

What is preferred way to do it? via command line parameter: default: white, --dark-error - dark?

@iansu
Copy link
Contributor

iansu commented Apr 5, 2019

No, we specifically don't want to add any kind of parameter or config to enable this. The prefers-color-scheme: dark media query is set automatically based on their OS colour scheme so that's what we should use.

@nartoland
Copy link

Ok

@Fabianopb
Copy link
Contributor

Hi, this looks like a fun issue to work on. Is someone already doing it?
If not I'd like to grab!
Cheers!

@iansu
Copy link
Contributor

iansu commented Apr 7, 2019

@Fabianopb It's all yours!

@kostia-lev
Copy link

interestingly, how to develop anything in packages/react-error-overlay folder? any instruction or helpful links?

@Nufflee
Copy link

Nufflee commented May 10, 2019

Any updates on this issue? @Fabianopb

@Fabianopb
Copy link
Contributor

I've had a busy beginning of month, but the work is half way through, I'll get back to it during the weekend!

@Fabianopb
Copy link
Contributor

Just created the PR 🚀

@Nufflee
Copy link

Nufflee commented May 15, 2019

That was really fast, thanks! Can't wait to try it out.

@William-Owen
Copy link
Author

Fantastic thank you Fabianopb!

@lock lock bot locked and limited conversation to collaborators Oct 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants