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.

Sign up for GitHub

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

Jump to bottom

Adjust color for OuiButtonGroup to improve contrast on page background #954

Closed
Tracked by #895
KrooshalUX opened this issue Aug 9, 2023 · 3 comments
Closed
Tracked by #895

Adjust color for OuiButtonGroup to improve contrast on page background #954

KrooshalUX opened this issue Aug 9, 2023 · 3 comments
Assignees
@BSFishy
Labels
next-theme

Comments

@KrooshalUX
Copy link
Contributor

No description provided.

@KrooshalUX KrooshalUX mentioned this issue Aug 9, 2023
Open
32 tasks
@KrooshalUX
Copy link
Contributor Author

@joshuarrrr @BSFishy can you supply the color sass variable information that is being used for this component, and its mapping back to the base sass variables?

@joshuarrrr joshuarrrr moved this to Todo in Look & Feel Aug 9, 2023
@joshuarrrr joshuarrrr assigned BSFishy and unassigned BSFishy Aug 15, 2023
@BSFishy
Copy link
Contributor

BSFishy commented Aug 17, 2023

Which variant needs to be checked? A screenshot would be useful

@BSFishy BSFishy moved this from Todo to UX Required in Look & Feel Aug 17, 2023
@KrooshalUX
Copy link
Contributor Author

Based on the docs site - it looks ok - but there is somewhere in the application it looked incorrect / too low contrast. I will have to find it again and open the issue related to the page it is on instead. Closing this for now.

@github-project-automation github-project-automation bot moved this from UX Required to Done in Look & Feel Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BSFishy BSFishy

Labels
next-theme
Projects
Look & Feel
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@joshuarrrr @BSFishy @KrooshalUX