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

Refactor/theme palette #443

Merged
merged 5 commits into from May 1, 2022
Merged

Refactor/theme palette #443

merged 5 commits into from May 1, 2022

Conversation

jrgarciadev
Copy link
Member

@jrgarciadev jrgarciadev commented May 1, 2022

📝 Description

A new theme palette was implemented because the old one doesn't support accessibility contrast, all components and docs were modified to use the new palette based on the radix-ui colors package https://www.radix-ui.com/colors

⛳️ Current behavior (updates)

The current theme palette has some accessibility contrast issues

🚀 New behavior

Accessible base theme palette with the right contrast and better scales starting at 50 and ending at 900, based on the radix-ui colors package https://www.radix-ui.com/colors

💣 Is this a breaking change (Yes/No): Yes

If you are using any of the NextUI base colors or accents colors through the tokens or theme object you should see and migrate your tokens/variables, takes this into account:

  • 500 step is now 600 for base colors ex: yellow500 (old) -> yellow600 (new)
  • accent1 is now accent0 an so on, ex: accent1 (old) -> accent0 (new)

📝 Additional Information

Users will be able to import the colors separately as follows:

import { blue, yellow, blueDark, yellowDark, Text } from "@nextui-org/react";

export default function App() {
  return (
    <div>
      <Text color={blue.blue800}>Option A</Text>
    </div>
  );
}

@jrgarciadev jrgarciadev added this to the 🔥 v1.0.0-beta.7 milestone May 1, 2022
@jrgarciadev jrgarciadev merged commit 9dc5d41 into next May 1, 2022
@jrgarciadev jrgarciadev deleted the refactor/theme-palette branch March 20, 2023 13:29
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.

None yet

1 participant