Skip to content

using virtualtext as one option for highlight#55

Open
DarwinSenior wants to merge 2 commits into
norcalli:masterfrom
DarwinSenior:master
Open

using virtualtext as one option for highlight#55
DarwinSenior wants to merge 2 commits into
norcalli:masterfrom
DarwinSenior:master

Conversation

@DarwinSenior

Copy link
Copy Markdown

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

@cseelus

cseelus commented Jul 13, 2021

Copy link
Copy Markdown

Works really well so far.

Screen Shot 2021-07-13 at 12 25 36

@norcalli: Can this be merged?

@mawkler

mawkler commented Oct 19, 2021

Copy link
Copy Markdown

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

@Mange

Mange commented Oct 20, 2021

Copy link
Copy Markdown

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

Comment thread lua/colorizer.lua
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 = '■';

Copy link
Copy Markdown

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

qbedard commented Feb 16, 2022

Copy link
Copy Markdown

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
Copy Markdown

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

@polyzen

polyzen commented Feb 27, 2023

Copy link
Copy Markdown

@manankarnik

Copy link
Copy Markdown

@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