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

Editor Notification Background Color #61

Conversation

arcticicestudio
Copy link
Contributor

@arcticicestudio arcticicestudio commented Jun 13, 2019

Resolves #52

Before the new UI theme API was released the editor color scheme provided multiple theme keys to also style some UI elements like the editor notifications (NOTIFICATION_BACKGROUND like also defined in the default bundled Darcula editor color scheme).
The key allows to style the background color of notifications from the editor that are placed right below the tab bar at the top of the editor.

Previously nord4 was used that made the text almost unreadable because by default a low contrast foreground is used.
This has now been changed to use nord3 making the actual text content readable again.

Before

After

The editor color scheme provides the `NOTIFICATION_BACKGROUND` key that
allows to style the background color of notifications from the editor
that are placed right below the tab bar at the top of the editor.
Previously `nord4` was used that made the color almost unreadable since
is uses a lower contrast by default instead of pure black.
This has now been changed to use `nord3` making the actual content
readable again.

GH-52
Copy link

@ice1000 ice1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ice1000
Copy link

ice1000 commented Jun 14, 2019

When can we see an update on this?

@arcticicestudio arcticicestudio merged commit 0bad27e into develop Jun 15, 2019
@arcticicestudio arcticicestudio deleted the improvement/gh-52-editor-notification-background-color branch June 15, 2019 08:29
@arcticicestudio arcticicestudio removed their assignment Jun 15, 2019
@arcticicestudio
Copy link
Contributor Author

@ice1000 Since my time to spend for OSS is very limited (and not payed), I had no time to write the release notes and deploy v0.7.0 to the JetBrains Plugin registry. The problem this PR solved also annoys me during my day job and private coding so I'll try to push the artifact today. It'll take ~12-24h to be globally available since plugin updates need to run through JetBrains verification process before being set to "confirmed" status.

@ice1000
Copy link

ice1000 commented Jul 16, 2019

Nice!

@arcticicestudio
Copy link
Contributor Author

Version 0.7.0 has been released and deployed to the Plugin Registry and will be available within the next 24h.

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

Successfully merging this pull request may close these issues.

Background color and text in tip "Gradle files have changed ..." are similar
3 participants