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

Default colors for errors is illegible #2901

Closed
Coding-Badly opened this issue Apr 4, 2015 · 13 comments
Closed

Default colors for errors is illegible #2901

Coding-Badly opened this issue Apr 4, 2015 · 13 comments
Milestone

Comments

@Coding-Badly
Copy link

There are numerous complaints about the dark-red text on black background colors for error messages. For good reason. The error messages are ILLEGIBLE.

@rogerclarkmelbourne
Copy link

I agree. I have to highlight them to read them. Dark red on a black background is terrible UI design

@mschlenker
Copy link
Contributor

How about the possibility to set some colors in some config file or as an environment variable? This way we could play around with some color combinations before settling?

BTW: I am making lots of screenshots for books, there dark text on light background looks much better than the other way round, especially in books printed in black and white.

@PaulStoffregen
Copy link
Contributor

How about the possibility to set some colors in some config file

Like lib/theme/theme.txt ?

@Coding-Badly
Copy link
Author

Paul, thank you. I could not remember the filename and searching through Google only led me to preferences.txt.

@Coding-Badly
Copy link
Author

In my opinion, the saturation for #A61717 is too low. Making it bright enough to be visible washes out the color. #FF1717 is much more legible.

@PaulStoffregen
Copy link
Contributor

For Teensy, I'm patching it to be #FF6060, which looks really good.

@mschlenker
Copy link
Contributor

Why not go with #FFFF00?

I basically agree with #FF6060. The bikeshed has a nice new color now.

@PaulStoffregen
Copy link
Contributor

Everyone can paint their lib/theme/theme.txt bikeshed any color they like.

@PaulStoffregen
Copy link
Contributor

But you did ask why. Well, in case you're not familiar with Teensy, I publish an installer which adds quite a large number of patches to Arduino. Teensy has a good number of features that aren't supported by the stock Arduino software, particularly relating to non-Serial USB modes.

My general policy is to avoid changing the IDE's behavior for non-Teensy boards, even when it might be beneficial. Instead, I take the long road of contributing improvements to Arduino which eventually become part of the official IDE. But sometimes, like this color change, adding code to preserve the original color when a non-Teensy board is selected is just too much work.

So I chose the brightest color that still has some resemblance to the original. Changing it to bright yellow might be even better from a readability point of view. But from a "why did your installer change the color" point of view, that's a pretty dramatic change that shouldn't be done by an installer like mine that modifies a user's existing copy.

@mschlenker
Copy link
Contributor

Paul, I completely agree with you. I think it's best to use red to indicate error - and then, choose a bright red. For personal settings like mine, reverse colors (for the books) or bright yellow or bright orange on black (working in the night) probably will do the job.

I am currently working on some Yún related project that probably will have some trickle down effects on Arduino (.cc's) Yún shield. And there some questions of design choice raise. And every suggestion has to be made carefully not breaking compatibility to older firmware and Doghunter boards. So discussing colors is just fun.

Thanks for hinting me to the appropriate location!

@00alis
Copy link
Contributor

00alis commented Apr 7, 2015

We reverted the Error color to the original one, see 8020c0d.
Fix available in the Nightly Build from tomorrow on, will be included in the next release.

Thanks for your feedback!

@ffissore ffissore closed this as completed Apr 7, 2015
@Coding-Badly
Copy link
Author

Huh. A burnt orange. You must be a Longhorn fan.

That is much better. Thanks.

@lmihalkovic
Copy link

see #4151 / #4158

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

No branches or pull requests

7 participants