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

feat: use kitty image protocol in ghostty #405

Merged
merged 1 commit into from
Dec 26, 2024
Merged

Conversation

mfontanini
Copy link
Owner

This detects ghostty and uses the kitty graphics protocol on it by default. I don't have access to ghostty yet so I can't test this but based on this article TERM_PROGRAM=ghostty is the way to detect this. If it implements the protocol correctly, this should work out of the box.

@fredrikaverpil could I ask for your help here? :)

@fredrikaverpil

This comment was marked as resolved.

@mfontanini
Copy link
Owner Author

Hmm interesting... I'll have to wait until it's publicly available then :(. Thanks for trying it out!

@fredrikaverpil
Copy link

@mfontanini it's out now 😄
http://ghostty.org

@mfontanini
Copy link
Owner Author

Looks like this was working already, merging!

@mfontanini mfontanini merged commit 5e651f6 into master Dec 26, 2024
6 checks passed
@mfontanini mfontanini deleted the feat/ghostty-images branch December 26, 2024 23:26
@fredrikaverpil

This comment was marked as resolved.

@mfontanini
Copy link
Owner Author

mfontanini commented Dec 27, 2024

Your initial command you posted a while back is just doing an install without this specific branch so this wouldn't have worked. Did you just install the latest from the master branch? Make sure you reinstall if you have it installed already.

But yes, images work as expected on ghostty for me.

@fredrikaverpil
Copy link

fredrikaverpil commented Dec 27, 2024

@mfontanini I just realized my mistake earlier, went to build off master, and came back to see you realized it too! 😆

It's all good (sorry for the noise):

image

Many thanks for your hard work on presenterm! ❤️

@mfontanini
Copy link
Owner Author

Awesome, thanks for verifying!

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