Skip to content

feat(catppuccin): remove all styles of @property#1240

Merged
ayamir merged 1 commit intomainfrom
fix/catppuccin-properties-hl
Apr 22, 2024
Merged

feat(catppuccin): remove all styles of @property#1240
ayamir merged 1 commit intomainfrom
fix/catppuccin-properties-hl

Conversation

@Jint-lzxy
Copy link
Collaborator

Based on the upstream spec, @property is typically associated with the "key" in key/value pairs. So IMO since we don't give special treatment to the "values" (such as assigning them an "italic" property), we should treat the "keys" in the same manner as well.

Based on the upstream spec, `@property` is typically associated with the "key"
in key/value pairs. So IMO since we don't give special treatment to the "values"
(such as assigning them an "italic" property), we should treat the "keys" in the
same manner as well.

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
Copy link
Collaborator

@charliie-dev charliie-dev left a comment

Choose a reason for hiding this comment

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

lgtm

@ayamir ayamir merged commit 092b5a0 into main Apr 22, 2024
@ayamir ayamir deleted the fix/catppuccin-properties-hl branch April 22, 2024 14:21
jczhang02 pushed a commit to jczhang02/nvimdots that referenced this pull request May 1, 2024
Based on the upstream spec, `@property` is typically associated with the "key"
in key/value pairs. So IMO since we don't give special treatment to the "values"
(such as assigning them an "italic" property), we should treat the "keys" in the
same manner as well.

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
Deg2v pushed a commit to Deg2v/nvimdots that referenced this pull request Jan 16, 2025
Based on the upstream spec, `@property` is typically associated with the "key"
in key/value pairs. So IMO since we don't give special treatment to the "values"
(such as assigning them an "italic" property), we should treat the "keys" in the
same manner as well.

Signed-off-by: Jint-lzxy <50296129+Jint-lzxy@users.noreply.github.com>
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.

3 participants