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

Experimental cursor highlighting support #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

norcalli
Copy link
Owner

@norcalli norcalli commented Oct 19, 2019

Add experimental, but functional implementation

@norcalli norcalli changed the base branch from master to fine-grained-control October 19, 2019 04:40
@norcalli norcalli changed the title Update algorithm and refactor. Experimental cursor highlighting support Oct 19, 2019
@norcalli norcalli changed the base branch from fine-grained-control to master October 19, 2019 04:41
@norcalli norcalli changed the base branch from master to fine-grained-control October 19, 2019 04:42
@norcalli norcalli mentioned this pull request Oct 19, 2019
@norcalli norcalli changed the base branch from fine-grained-control to master October 19, 2019 04:52
@doronbehar
Copy link

Did you test it with GUI? I can see the echo command printing the cursor position etc but my cursor doesn't change it's color. I can run hi Cursor while hovering on white / black color texts and see that it's changing.

I suspect neovim/neovim#6591 is the reason and that it marks this and #1 as "won't fix".

@doronbehar
Copy link

BTW thank you so much for attending this so quickly and for being responsive and kind! I really appreciate it ☺️.

@norcalli
Copy link
Owner Author

@doronbehar Thank you for the kind words :) I'll leave this open in case the situation changes and we can support it one day.

@doronbehar
Copy link

Cool, I'm subscribed to neovim/neovim#6591 so in case it's fixed I'll check whether this PR works.

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