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

feat(claude): init #1579

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

Conversation

ruiiiijiiiiang
Copy link

@ruiiiijiiiiang ruiiiijiiiiang commented Jan 14, 2025

🎉 Theme for Claude 🎉

Claude.ai, similar to ChatGPT, is an AI-powered chatbot and assistant developed by Anthropic, designed to provide conversational, creative, and problem-solving support. It focuses on generating helpful, safe, and user-aligned responses.

Latte with Peach

2025-01-14-121025_hyprshot

Frappe with Sapphire

2025-01-14-121535_hyprshot

Macchiato with Flamingo

2025-01-14-121625_hyprshot

Mocha with Mauve

2025-01-14-121421_hyprshot

🗒 Checklist 🗒

  • I have read and followed Catppuccin's submission guidelines.
  • I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.
    • I have ensured that the new directory is in lower-kebab-case.
    • I have followed the template and kept the preprocessor as LESS.
  • I have made sure to update the userstyles.yml file with information about the new userstyle.
  • I have included the following files:
    • catppuccin.user.less - all the CSS for the userstyle, based on the template.
    • preview.webp - composite image of all four individual flavor screenshots (taken with the default accent color of mauve) stitched together, generated via Catwalk.

@ruiiiijiiiiang ruiiiijiiiiang changed the title Feat/claude feat(claude): init Jan 14, 2025
scripts/userstyles.yml Outdated Show resolved Hide resolved
styles/claude/catppuccin.user.less Outdated Show resolved Hide resolved
styles/claude/catppuccin.user.less Outdated Show resolved Hide resolved
styles/claude/catppuccin.user.less Outdated Show resolved Hide resolved
--accent-pro-100: #hslify(@accent) [];
--accent-pro-200: #hslify(@accent) [];
--accent-pro-900: #hslify(lighten(@accent, 10%)) [];
--accent-pro-900: #hslify(darken(@accent, 25%)) [];
Copy link
Member

Choose a reason for hiding this comment

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

We should avoid darkening colors by more than 10% if possible.

Copy link
Author

Choose a reason for hiding this comment

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

I was hoping to create more contrast for elements like this one, where the background color is accent-pro-900. If I change the darken ratio to 10%, the text becomes very hard to read:

image

What's a good approach to make it more readable without too much darkening?

Copy link
Member

Choose a reason for hiding this comment

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

Invert the text color to base/mantle/crust.

Copy link
Author

Choose a reason for hiding this comment

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

I added a special rule so I didn't have to change the color for text-300.

Copy link
Member

@uncenter uncenter left a comment

Choose a reason for hiding this comment

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

Login page unthemed:
CleanShot 2025-01-31 at 21 29 09

Comment on lines +122 to +145
/* Generated code in green */
span.token[style="color: rgb(152, 195, 121);"] {
color: @green !important;
}

/* Generated code in yellow */
span.token[style="color: rgb(209, 154, 102);"] {
color: @yellow !important;
}

/* Generated code in blue */
span.token[style="color: rgb(97, 175, 239);"] {
color: @blue !important;
}

/* Generated code in red */
span.token[style="color: rgb(224, 108, 117);"] {
color: @red !important;
}

/* Generated code in magenta */
span.token[style="color: rgb(198, 120, 221);"] {
color: @mauve !important;
}
Copy link
Member

Choose a reason for hiding this comment

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

We should ideally try to match this to our own syntax highlighting, but if these inline styles are all we got... I guess not.

Copy link
Author

Choose a reason for hiding this comment

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

Yeah I wish I could do that but unfortunately this is all I have to work with 😓

@uncenter
Copy link
Member

uncenter commented Feb 1, 2025

CleanShot 2025-01-31 at 21 31 05

Boxed in red are areas of text that I can tell are unthemed based on the preview image.

@uncenter uncenter added the waiting on author Note for staff that a re-review is not yet required label Feb 1, 2025
@ruiiiijiiiiang
Copy link
Author

@uncenter The onboarding page is now being styled correctly.
The unstyled text in the last preview was from the first iteration. It has been fixed. This is the latest preview:
preview

@uncenter uncenter removed the waiting on author Note for staff that a re-review is not yet required label Feb 2, 2025
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.

2 participants