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

Update value ouiColorPrimary in Next Light theme #979

Closed
Tracked by #895
KrooshalUX opened this issue Aug 18, 2023 · 2 comments · Fixed by #981
Closed
Tracked by #895

Update value ouiColorPrimary in Next Light theme #979

KrooshalUX opened this issue Aug 18, 2023 · 2 comments · Fixed by #981

Comments

@KrooshalUX
Copy link
Contributor

KrooshalUX commented Aug 18, 2023

Update value from #159D8D to #07827E in Light mode variation of Next theme. Do not change Primary definition for Dark mode in Next theme.

This should significantly improve calculated contrast ratios and does not rely on font weight to pass.

@BSFishy
Copy link
Contributor

BSFishy commented Aug 22, 2023

Addressed in #981. Still very close to success, but fixes color system issue

@KrooshalUX
Copy link
Contributor Author

@BSFishy we will evolve success, danger and warning separately, but understood.

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 a pull request may close this issue.

2 participants