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

Accent MatchParen when cursor is not blinking. #14

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

Conversation

shofel
Copy link

@shofel shofel commented Jul 4, 2018

I believe, this change is almost unopinionated, but just makes the things better :)

The Problem

When for MatchParen fg=syntax_bg, then block-shaped cursor becomes colored with bg_color, and
becomes invisible. This makes "another" paren much more accented than "this" paren.

Fix

To fix this, I just swap bg and fg for MatchParen, which as for me, solves the problem.

Demo video

After watching the demo I've realized, the old variant looks better when the cursor is blinking.
For other cursor shapes (horizontal line, vertical line, non-blinking block) new version looks and feels a way better.

Demo file

/**
 * Some parens to walk around.
 */
let f = ([x, y, {z: za}]) => ({ xs: [x, y], za })

/**
 * Vim commands to set different types of cursors (for neovim)
 */

/* block */
set guicursor=n:block

/* blinking block */
set guicursor=n:block-blinkoff400-blinkon250

/* horizontal line */
set guicursor=n:hor100

/* vertical line */
set guicursor=n:ver100

The problem: when for MatchParen fg=syntax_bg, then block-shaped cursor coloured with bg color,
that is become invisible. This makes "another" paren more accented than "this" paren.

To fix this, I just swap bg and fg for MatchParen, which solves the problem.
@shofel shofel changed the title accent MatchParen; subtle SignColumn Accent MatchParen when cursor is not blinking. Jul 4, 2018
@luxmeter
Copy link

OH GOD I THANK YOU SO MUCH

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