Skip to content

fix(catppuccin.ts): borders from black -> overlay0#38

Merged
sgoudham merged 3 commits intocatppuccin:mainfrom
ninetailedtori:panel-borders
Apr 10, 2026
Merged

fix(catppuccin.ts): borders from black -> overlay0#38
sgoudham merged 3 commits intocatppuccin:mainfrom
ninetailedtori:panel-borders

Conversation

@ninetailedtori
Copy link
Copy Markdown
Contributor

No description provided.

ninetailedtori added a commit to ninetailedtori/catppuccin-codemirror that referenced this pull request Apr 9, 2026
Yes, it breaks our css and thus will be ENTIRELY IGNORED but we'll change it in another PR.
C.F. catppuccin#38

Signed-off-by: Toria <ninetailedtori@uwu.gal>
Copy link
Copy Markdown
Contributor

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

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

I'm surprised we even set these borders, what happens if you don't set this CSS in the first place? What does it look like?

@ninetailedtori
Copy link
Copy Markdown
Contributor Author

ninetailedtori commented Apr 9, 2026

I'm surprised we even set these borders, what happens if you don't set this CSS in the first place? What does it look like?

https://codemirror.net/docs/ref/#h_panels here's the ref for panels! Honestly, they DO make a difference, lemme show you the original!

image

@sgoudham
Copy link
Copy Markdown
Contributor

How does it look with overlay0?

@ninetailedtori
Copy link
Copy Markdown
Contributor Author

ninetailedtori commented Apr 10, 2026

image

It's alright! But honestly we could...just make it invisible? Or even, I think, undefine it? I'm not sure what upstream sets for that!

@ninetailedtori
Copy link
Copy Markdown
Contributor Author

image

No border does make it look...unclear though. Perhaps we should make that a...1px border?

@sgoudham
Copy link
Copy Markdown
Contributor

If we want to keep the border, 1px overlay0 seems like it'd be good

@ninetailedtori
Copy link
Copy Markdown
Contributor Author

If we want to keep the border, 1px overlay0 seems like it'd be good

Sounds good!

@sgoudham sgoudham merged commit 96a620d into catppuccin:main Apr 10, 2026
1 check passed
@ninetailedtori ninetailedtori deleted the panel-borders branch April 10, 2026 01:35
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