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

Issue 301 - Increase contrast between P1 and P2 labels #315

Merged
merged 2 commits into from
May 7, 2024

Conversation

sarahhjchung
Copy link
Collaborator

@sarahhjchung sarahhjchung commented May 3, 2024

Fixes #301

These are the screenshots showing the new colors chosen for the priority labels. The colors have been adjusted to work in both light and dark mode (using dark reader).

Screenshot 2024-05-06 at 10 16 07 AM Screenshot 2024-05-06 at 10 15 55 AM

@aminomancer
Copy link
Member

aminomancer commented May 4, 2024

Remember to add

Fixes #301

to your pull request descriptions or extended commit messages, it helps with tracking and it'll automatically close the issue when the PR merges.

Btw, I use dark reader so I judge the colors by how well they work in both light mode & dark reader haha. So you may want to test with both light & dark also.

border-color: transparent;
color: white;
}
.p2 {
background: #9fc6ee;
background: #9FC6EE;
Copy link
Member

Choose a reason for hiding this comment

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

I'm confused. These are the same colors, just in uppercase?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sorry I don't know what happened to the modified colors but I'll update the colors again anyway to work in both light and dark mode.

Copy link
Member

@aminomancer aminomancer left a comment

Choose a reason for hiding this comment

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

Purple looks great, thank you!

@aminomancer
Copy link
Member

Feel free to merge - make sure to hit "Squash and merge" and not the other options.

@sarahhjchung sarahhjchung merged commit 6abfbbd into mozilla:main May 7, 2024
@sarahhjchung sarahhjchung deleted the issue301 branch May 7, 2024 13:41
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

Successfully merging this pull request may close these issues.

[BUG] Not enough contrast between P1 and P2 labels
2 participants