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

using virtualtext as one option for highlight #55

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

Conversation

DarwinSenior
Copy link

Taking the inspiration from vim-hexokinase. Adding virtual text as one other option to create virtual text display mode.

@cseelus
Copy link

cseelus commented Jul 13, 2021

Works really well so far.

Screen Shot 2021-07-13 at 12 25 36

@norcalli: Can this be merged?

@mawkler
Copy link

mawkler commented Oct 19, 2021

Any update on this? Is it ready to be merged?

@Mange
Copy link

Mange commented Oct 20, 2021

I've been this daily for a long time, no issues found so far.

Comment on lines +59 to +69
RGB = true; -- #RGB hex codes
RRGGBB = true; -- #RRGGBB hex codes
names = true; -- "Name" codes like Blue
RRGGBBAA = false; -- #RRGGBBAA hex codes
rgb_fn = false; -- CSS rgb() and rgba() functions
hsl_fn = false; -- CSS hsl() and hsla() functions
css = false; -- Enable all CSS features: rgb_fn, hsl_fn, names, RGB, RRGGBB
css_fn = false; -- Enable all CSS *functions*: rgb_fn, hsl_fn
-- Available modes: foreground, background, sign, virtualtext
mode = 'background'; -- Set the display mode.
virtualtext = '■';

Choose a reason for hiding this comment

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

it seems a little dis-aligned, doesn't it?

@qbedard
Copy link

qbedard commented Feb 16, 2022

Doesn't look like it works when setting the mode via default_options. Need to update the validation there to allow "virtualtext" as a value.

@manankarnik
Copy link

Thank you for this, works fine for me. Why isn't this merged?

@polyzen
Copy link

polyzen commented Feb 27, 2023

@manankarnik
Copy link

@polyzen, Looks like this repo isn't maintained anymore. Would be nice if there was some indication of that (other than the last commit date) or the repo was archived. Anyway, thanks for linking a maintained fork 😁

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.

8 participants