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

Improved color bindings and support for all keywords of current i3 specifications. #38

Open
wants to merge 39 commits into
base: master
Choose a base branch
from

Conversation

litoj
Copy link

@litoj litoj commented Aug 6, 2021

Hello,
I would like to contribute my additions to the syntax to support all keywords i3 has right now.
I also notably changed color bindings to provide a more pg-language-like experience. More color links are used and newly I added a better highlighting conditions.
This makes it possible for more colors to be used and people can distinguish more different functions.

@litoj
Copy link
Author

litoj commented Aug 6, 2021

The edits in README are mainly for myself, those were not supposed to go to the request, but I don't know how to remove a file from the request.

@hiqua
Copy link

hiqua commented Dec 14, 2021

Please consider making changes to the upstream vim repo directly as this code is there as of vim/vim@8176be1

@hiqua
Copy link

hiqua commented Jun 16, 2022

@JosefLitos offer still stands to commit to the vim repo directly and make your changes available to everyone, including every other Arch Linux users.

@litoj
Copy link
Author

litoj commented Jun 17, 2022

I may give it a try soon, if all goes as planned.

@litoj
Copy link
Author

litoj commented Jun 20, 2022

@hiqua thanks for reminding me, should be ready to go, just made a couple more fixes:
vim/vim#10600

@litoj litoj force-pushed the master branch 2 times, most recently from de445d8 to d7b0c6e Compare September 9, 2023 11:23
@litoj litoj force-pushed the master branch 2 times, most recently from f76db58 to badbf26 Compare September 15, 2023 06:54
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