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

updated globals colors #107

Merged
merged 1 commit into from
Oct 29, 2024
Merged

updated globals colors #107

merged 1 commit into from
Oct 29, 2024

Conversation

skull8888888
Copy link
Collaborator

@skull8888888 skull8888888 commented Oct 29, 2024

Important

Updated color variables in globals.css and adjusted CodeEditor component to use new colors.

  • CSS Variables:
    • Updated color variables in globals.css for --background, --foreground, --card, --popover, --secondary, and their foreground counterparts.
  • Code Editor Component:
    • Changed bg-secondary to bg-card in code-editor.tsx for the div and CodeMirror component.

This description was created by Ellipsis for dbe762d. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to dbe762d in 7 seconds

More details
  • Looked at 50 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. frontend/components/ui/code-editor.tsx:67
  • Draft comment:
    Ensure consistency in class names. Consider updating all instances of 'bg-secondary' to 'bg-card' if applicable.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The change from 'bg-secondary' to 'bg-card' in the CodeEditor component should be consistent across all related class names.

Workflow ID: wflow_Brt2VBtvAoDO2qJB


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@skull8888888 skull8888888 merged commit 599a3d5 into dev Oct 29, 2024
1 check passed
@skull8888888 skull8888888 deleted the colors branch October 29, 2024 06:38
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 this pull request may close these issues.

1 participant