Skip to content

Conversation

@BuonOmo
Copy link
Contributor

@BuonOmo BuonOmo commented Mar 27, 2025

CodeMirror 6 changes from CSS to JavaScript stylesheets. This breaks compatiblity with CodeMirror 5. The new architecture is based on the one-dark example theme from the official CodeMirror repository.

Fixes #2

CodeMirror 6 changes from CSS to JavaScript stylesheets.
This breaks compatiblity with CodeMirror 5. The new
architecture is based on the one-dark example theme
from the official CodeMirror repository.

Fixes catppuccin#2
package.json Outdated
"version": "6.0.0",
"description": "Catppuccin themes for the CodeMirror code editor",
"scripts": {
"test": "echo 'Life is beautiful.'",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"test": "echo 'Life is beautiful.'",

package.json Outdated
Comment on lines 13 to 17
"author": {
"name": "Ulysse Buonomo",
"email": "[email protected]",
"url": "https://ulysse.md"
},
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"author": {
"name": "Ulysse Buonomo",
"email": "[email protected]",
"url": "https://ulysse.md"
},
"author": "Catppuccin Org <[email protected]>",

@BuonOmo
Copy link
Contributor Author

BuonOmo commented Mar 28, 2025

@uncenter Thanks for the thorough review, would you mind reviewing the fixup commit?

@BuonOmo BuonOmo requested a review from uncenter April 17, 2025 13:29
@BuonOmo
Copy link
Contributor Author

BuonOmo commented Apr 17, 2025

Hey @uncenter, may I ask a bit of your time before this falls into oblivion?

Or maybe @sgoudham?

@sgoudham sgoudham requested review from sgoudham and removed request for uncenter April 17, 2025 13:34
@sgoudham
Copy link
Contributor

Hey 👋,

I want to confirm that I will be looking at this, just been in other areas of the organisation as of late. Your work won't go unnoticed / stale ❤️ @BuonOmo

Copy link
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.

Thank you so much for the PR! I still need to tidy some stuff up and set up a publishing workflow but I'd like to merge this and iterate on the main branch.

@sgoudham sgoudham merged commit 949d7d0 into catppuccin:main Apr 27, 2025
@BuonOmo BuonOmo deleted the feat/codemirror6 branch April 28, 2025 06:19
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.

Add support for v6 (latest)

3 participants