Skip to content

fix(claude): syntax highlighting#2233

Open
craciu25yt wants to merge 3 commits intocatppuccin:mainfrom
craciu25yt:patch-1
Open

fix(claude): syntax highlighting#2233
craciu25yt wants to merge 3 commits intocatppuccin:mainfrom
craciu25yt:patch-1

Conversation

@craciu25yt
Copy link
Copy Markdown

Claude codeblocks was broken and didn't show up syntaxt highlight colours
Default:
{649A716E-4313-4871-95BE-1304EDDB1240}

Catppuccini:
{94629C77-88B7-4A1C-932C-790B17F69ADA}

Fixed:
image

@github-actions github-actions Bot added the claude Claude label Apr 26, 2026
Comment thread styles/claude/catppuccin.user.less Outdated
@uncenter uncenter changed the title Fixes codeblocks colour highlights fix(claude): syntax highlighting Apr 26, 2026
Co-authored-by: uncenter <uncenter@uncenter.dev>
@craciu25yt
Copy link
Copy Markdown
Author

my bad, didn't know

@uncenter
Copy link
Copy Markdown
Member

my bad, didn't know

No worries! See our style guide: https://github.com/catppuccin/catppuccin/blob/main/docs/style-guide.md#code-editors.

/* Code blocks */
.code-block__code {
background: @crust !important;
border-radius: 0px 0px 8px 8px !important;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please don't set non-color properties. As outlined in our contributing guidelines, we prohibit opinionated changes. https://userstyles.catppuccin.com/contributing/#opinionated-changes

Suggested change
border-radius: 0px 0px 8px 8px !important;

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

that was because if not the colour was not displayed correctly
{80CE8B94-8505-4474-AF43-06685233D8A7}
{8F241401-E023-4B89-87A7-6190DAE0610F}

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I don't think thats opinated, its just filling the box as it shows up
{1F7A2708-EE7F-4D41-8F9D-4E4914EAA266}
normally

Comment on lines +94 to +100
/* Target the specific element with all those classes */

.text-text-500.font-small.p-3\.5.pb-0 {
background-color: @crust !important;
/* Optional: Add padding or border-radius if the bg looks cramped */
border-radius: 8px 8px 0px 0px !important;
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
/* Target the specific element with all those classes */
.text-text-500.font-small.p-3\.5.pb-0 {
background-color: @crust !important;
/* Optional: Add padding or border-radius if the bg looks cramped */
border-radius: 8px 8px 0px 0px !important;
}

I'm not sure what this is for. Please isolate this to a separate pull request (this one should just be for the syntax highlighting fix) and provide a clearer description of where this is and what it changes - can we get a screenshot of what it looked like before this change and what it looks like after?

Copy link
Copy Markdown
Author

@craciu25yt craciu25yt May 2, 2026

Choose a reason for hiding this comment

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

that is also realted to the upper discussion
{6299E0FE-CEE6-4873-AA15-CE2A76404C24}
{C557F63A-7F26-4D46-8009-B5F2013203BF}
these two are removing .text-text-500.font-small.p-3\.5.pb-0

image this one `.code-block`'s `border-radius: 0px 0px 8px 8px !important; ` (a slight radius is seen, making it seem weird)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

maybe the way I've done it its not the best as I'm not very good at css so any improvement feedback is accepted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude Claude

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants