Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port Cyberdream to Helix editor #28

Merged
merged 2 commits into from
May 3, 2024

Conversation

zoedsoupe
Copy link
Contributor

@zoedsoupe zoedsoupe commented Apr 30, 2024

helix is one of more recent modal editors, based on kakoune. It has builtin support to treesitter and LSP.

I tried to port it but i think the theme config needs some improvements as it isn't complete match with the original. What do you think? Could anyone help me?

image image

the complete theme creation documentation of helix can be found here: https://docs.helix-editor.com/themes.html

@zoedsoupe
Copy link
Contributor Author

it seems that the cursor has the same color of the background, causing it to be "invisible"

@scottmckendry scottmckendry added enhancement New feature or request help wanted Extra attention is needed labels Apr 30, 2024
@scottmckendry
Copy link
Owner

Thanks for opening the PR! Looking great so far 🙂
I'll take a look at the theme when I next get a chance. Never used Helix but keen to take a look.

I encourage anyone who is actively using Helix to give it a spin too and provide feedback.

@scottmckendry
Copy link
Owner

I've made a few changes and now it looks almost perfect (at least when looking at Golang code). Here's a screenshot of Helix at the top and nvim at the bottom:

image

Thanks again for the contribution!

@scottmckendry scottmckendry merged commit 2420116 into scottmckendry:main May 3, 2024
@zoedsoupe zoedsoupe deleted the feat/helix-theme branch May 7, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants