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

Wezterm support #17

Open
adelarsq opened this issue Aug 17, 2022 · 5 comments
Open

Wezterm support #17

adelarsq opened this issue Aug 17, 2022 · 5 comments

Comments

@adelarsq
Copy link

Now that Wezterm has some support for Kitty protocol I am trying to make this to work.

But for me shows this error for the minimal setup:

image

Will be needed something more on the plugin or terminal for this to work?

@OmeletWithoutEgg
Copy link
Contributor

You might need require('hologram').setup {} to prevent the error.
However, the protocol support of wezterm seems quite weird...
I can't get hologram.nvim work in wezterm.

@adelarsq
Copy link
Author

You might need require('hologram').setup {} to prevent the error.

I am already using but no way to make this work. 😢

@AnonymouX47
Copy link

Wezterm's current support for the protocol is still highly experimental.

From my personal experience, most features are either not yet implemented or not correctly implemented.

Support for the iTerm2 image protocol is stable though and might be a better option on Wezterm as soon as it's added here (i.e in hologram.nvim).

@optimizasean
Copy link

optimizasean commented Nov 14, 2022

@OmeletWithoutEgg

As a note, add this to your init.vim (~/.config/nvim/init.vim):

" Install Hologram with Plug
Plug 'edluffy/hologram.nvim'

" Setup Hologram
lua << EOF
require'hologram'.setup{
    auto_display = true -- WIP automatic markdown image display, may be prone to breaking
}
EOF

Will fail first time, once you run PlugUpdate after opening NeoVim then the error should be gone.
The parenthesis cause errors for some reason (not a lua specialist here).

It works in WezTerm on macOS but the image renders funky - correct, but on top of things (kind of) and small.
Not sure what is wrong with it.

Discovered bug on this png in WezTerm nightly with NeoVim latest: https://github.com/video-dev/hls.js/blob/master/docs/design.md, line 68.

EDIT:
Tested across number of images and types.
Apparently has different issues for each, not exactly sure of the relation.
Errors include: rendering over cursor, rendering at odd sizes (aspect correct, but not source image size or some fixed size), following cursor, overlaying over text, not in location (which is spaced correctly for size of image rendered), and a couple other weird bugs.
Potentially could be my environment but I would be very interested in seeing this work for WezTerm.

@AnonymouX47
Copy link

@optimizasean

I would be very interested in seeing this work for WezTerm.

This can only be achieved on WezTerm's end.

See wez/wezterm#986 (comment)

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

No branches or pull requests

4 participants