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

Feature: Update colors for for dark theme #1476 #1595

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Charlesnorris509
Copy link

Description:
This pull request addresses issue [#1476] by updating the color scheme for the dark theme to reduce bright colors and make the theme more suitable for dark mode. The changes include updating various color values in the Tailwind CSS configuration and the main CSS file.

### Changes Made:

_frontend/tailwind.config.cjs:
Updated the color values for different elements in the dark theme:
White text in dark theme: #E0E0E0
Modal background: #26272E
Table borders and other lines: #3D3D40
Input fields/dropdowns borders: #6A6A6A

frontend/src/index.css:
Updated the color values for different elements in the dark theme:
Background color for dark theme: #26272E
Text color for dark theme: #E0E0E0
Scrollbar track color for dark theme: #3D3D40
Table border and text color for dark theme: #3D3D40 and #E0E0E0
Input autofill background and text color for dark theme: #3D3D40 and #E0E0E0_

Copy link

vercel bot commented Jan 21, 2025

@Charlesnorris509 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ❌ Failed (Inspect) Jan 21, 2025 5:19pm

@dartpain
Copy link
Contributor

Hey, there seems to be a build error
error during build:
[vite:css] [postcss] /vercel/path0/frontend/src/index.css:54:5: The dark:border-3D3D40 class does not exist. If dark:border-3D3D40 is a custom class, make sure it is defined within a @layer directive.
file: /vercel/path0/frontend/src/index.css:54:4
at Input.error (/vercel/path0/frontend/node_modules/postcss/lib/input.js:106:16)
at AtRule.error (/vercel/path0/frontend/node_modules/postcss/lib/node.js:145:32)
at processApply (/vercel/path0/frontend/node_modules/tailwindcss/lib/lib/expandApplyAtRules.js:380:29)
at /vercel/path0/frontend/node_modules/tailwindcss/lib/lib/expandApplyAtRules.js:551:9
at /vercel/path0/frontend/node_modules/tailwindcss/lib/processTailwindFeatures.js:55:50
at async plugins (/vercel/path0/frontend/node_modules/tailwindcss/lib/plugin.js:38:17)
at async LazyResult.runAsync (/vercel/path0/frontend/node_modules/postcss/lib/lazy-result.js:261:11)
at async compileCSS (file:///vercel/path0/frontend/node_modules/vite/dist/node/chunks/dep-CB_7IfJ-.js:36897:21)
at async Object.transform (file:///vercel/path0/frontend/node_modules/vite/dist/node/chunks/dep-CB_7IfJ-.js:36170:11)
at async transform (file:///vercel/path0/frontend/node_modules/rollup/dist/es/shared/node-entry.js:18848:16)
Error: Command "npm run build" exited with 1

@Charlesnorris509
Copy link
Author

@dartpain would you be able to tell me whether the modifications made on the initial PR were accurate?

… and improve overall consistency. It updates various styles related to background colors, text colors, modal backgrounds, table borders, and input field borders as outlined in the referenced Figma design specs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants