-
Notifications
You must be signed in to change notification settings - Fork 59
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
Night mode audit #2118
Night mode audit #2118
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
9a8e1ec
to
48cf33a
Compare
@@ -22,6 +22,6 @@ | |||
|
|||
<style lang="postcss"> | |||
.frequency { | |||
@apply flex h-auto max-h-32 flex-col overflow-auto rounded-lg bg-inverse px-2 py-2 font-mono text-sm text-[#e9d5ff]; | |||
@apply flex h-auto max-h-32 flex-col overflow-auto rounded-lg bg-spaceBlack px-2 py-2 font-mono text-sm text-[#e9d5ff]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we just make spaceBlack
… black
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would but we also have this black that we use.
...rkflow/workflows-summary-configurable-table/tests/__snapshots__/table-body-cell.test.ts.snap
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is amazing @laurakwhit great work! I have some questions/comments but nothing major
Description & motivation 💭
Audit of color variables and Holocene components to align with designs.
Screenshots (if applicable) 📸
Workflows
Workflow detail
Timeline
Design Considerations 🎨
Testing 🧪
How was this tested 👻
Steps for others to test: 🚶🏽♂️🚶🏽♀️
Components with significant changes to verify look/work as expected:
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
DT-2137
Docs
Any docs updates needed?