Skip to content

fix(treesitter): set @type.builtin as italic#944

Merged
comfysage merged 1 commit into
catppuccin:mainfrom
igorlfs:fix/unused-fallback-styles
Feb 15, 2026
Merged

fix(treesitter): set @type.builtin as italic#944
comfysage merged 1 commit into
catppuccin:mainfrom
igorlfs:fix/unused-fallback-styles

Conversation

@igorlfs
Copy link
Copy Markdown
Contributor

@igorlfs igorlfs commented Oct 14, 2025

Hello,

styles.properties is set as {} in the default config, which is truthy in Lua. Therefore, to actually fall back to italic, users would have to set styles.properties to nil (which I'm assuming is unintended behavior).

Alternatively, @type.builtin could be italic by default.

@comfysage
Copy link
Copy Markdown
Collaborator

hi, great observation. could you - as you suggested - also set the default option for types to italic?

@igorlfs igorlfs force-pushed the fix/unused-fallback-styles branch from 372f8cf to c2ba250 Compare October 25, 2025 15:11
@igorlfs igorlfs changed the title fix(treesitter): do not set fallback for @type.builtin fix(treesitter): set @type.builtin as italic Oct 25, 2025
@igorlfs
Copy link
Copy Markdown
Contributor Author

igorlfs commented Oct 25, 2025

also set the default option for types to italic?

Done! I removed the O.styles.properties, not sure if keeping it makes much sense? Setting that to italic would obviously affect some other stuff.

@comfysage comfysage force-pushed the fix/unused-fallback-styles branch from c2ba250 to e5cdb39 Compare February 15, 2026 14:11
@comfysage comfysage merged commit 0a5de4d into catppuccin:main Feb 15, 2026
9 of 10 checks passed
@igorlfs igorlfs deleted the fix/unused-fallback-styles branch February 15, 2026 21:15
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