Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

Image support #239

Closed
w3bb opened this issue Jan 25, 2020 · 10 comments
Closed

Image support #239

w3bb opened this issue Jan 25, 2020 · 10 comments

Comments

@w3bb
Copy link

w3bb commented Jan 25, 2020

What do you want

Ability to display images using Ueberzug, w3m, or some other solution.

Why

It would be way less of a pain to have to open images by hand.

Implementation hints

Perhaps a solution would be to allow users to add some sort of image handling script like Ranger does?

@w3bb
Copy link
Author

w3bb commented Jan 25, 2020

Now that I think about it this could be done with the scripting interface. Have an event for images appearing, and an event for shifting the image up.

@Bios-Marcel
Copy link
Owner

Could you elaborate on that? What would the API look like?

@w3bb
Copy link
Author

w3bb commented Jan 25, 2020

The idea I had was for Cordless to call a script with parameters for the position of the image. However I didn't know the scripting interface existed and I'm not to familiar with it.

@Bios-Marcel
Copy link
Owner

yeah, dunno bout that. The scripting engine is a confined JS engine without access to the operating system that can only use specified APIs to interact with cordless.

@w3bb
Copy link
Author

w3bb commented Jan 25, 2020

Oh I figured it was shell script.

@Bios-Marcel
Copy link
Owner

I didn't want this to be something that people download scripts for that just bust their whole fucking system :)

@w3bb
Copy link
Author

w3bb commented Jan 26, 2020

It wouldn't bust their system since if they use a TUI they probably know how to be savvy, but maybe native support for w3m/ueberzug might work better?

@ChillerDragon
Copy link

ChillerDragon commented Jan 28, 2020

@Bios-Marcel
Copy link
Owner

@ChillerDragon That might actually be cool. I already considered that

@Bios-Marcel
Copy link
Owner

In all honestly, it doesn't make sense unless your images are always tiny. Because even a font-size of smth like 10px, which is already small, obviously takes more space than a pixel. Instead, more focus should be put on easily allowing to open links (images) outside of cordless.

Regarding ueberzug or anything. I think this kind of defies the laws of the terminal. It isn't crossplatform and won't even run in all linux terminals. Therefore I am not going introduce the complexity for such a tiny target audience.

Therefore, smth like #241 would be more interesting.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants