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

Fix colour keys in bridge #448

Merged
merged 1 commit into from
Jul 15, 2024
Merged

Fix colour keys in bridge #448

merged 1 commit into from
Jul 15, 2024

Conversation

rock3r
Copy link
Collaborator

@rock3r rock3r commented Jul 15, 2024

We had some en-GB key name spelling (Grey) which wasn't found at runtime since the actual palette uses en-US spelling (Gray). This PR fixes it. No issues were present in the standalone version. These cosmetic issues were until now masked by us providing defaults that the code falls back on when it can't load the keys; we had a warning in the logs but I suppose nobody paid attention to them since it all worked anyway 😅

We had some en-GB key name spelling (Grey) which wasn't found at runtime
since the actual palette uses en-US spelling (Gray). This PR fixes it.
@rock3r rock3r added bug Something isn't working bridge Something that impacts the Swing LaF bridge labels Jul 15, 2024
@rock3r rock3r requested a review from hamen July 15, 2024 13:03
@rock3r rock3r self-assigned this Jul 15, 2024
Copy link
Collaborator

@hamen hamen left a comment

Choose a reason for hiding this comment

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

IgnoreWarningAustinEvansGIF

@hamen hamen merged commit b6fe7ae into main Jul 15, 2024
4 checks passed
@hamen hamen deleted the fix-bridge-keys branch July 15, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bridge Something that impacts the Swing LaF bridge bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants