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

support image display for windows terminal #1522

Closed
2 of 3 tasks
echaya opened this issue Aug 19, 2024 · 8 comments
Closed
2 of 3 tasks

support image display for windows terminal #1522

echaya opened this issue Aug 19, 2024 · 8 comments
Labels
invalid This doesn't seem right

Comments

@echaya
Copy link

echaya commented Aug 19, 2024

yazi --debug output

.

Please describe the problem you're trying to solve

Hi,

I have only discovered yazi today (From reddit) and fall in love with it immediately!

Since Windows terminal is my daily drive, and per microsoft/terminal#448, sixel is supported natively by windows terminal, can we please consider support image please? Thanks in advance!!!

Would you be willing to contribute this feature?

  • Yes, I'll give it a shot

Describe the solution you'd like

support image view out of box for windows terminal

Additional context

No response

Validations

  • I have searched the existing issues
  • The latest nightly build of Yazi doesn't already have this feature
@echaya echaya added the feature New feature request label Aug 19, 2024
@sxyazi
Copy link
Owner

sxyazi commented Aug 19, 2024

Please do not skip yazi --debug, as I won't have enough information to handle the issue without it - I need to know what image protocol Yazi detects for MS Terminal and whether MS Terminal returns the correct DA1 response to indicate that it supports Sixel.

@sxyazi sxyazi added invalid This doesn't seem right waiting on op Waiting for more information from the original poster and removed feature New feature request labels Aug 19, 2024
@echaya
Copy link
Author

echaya commented Aug 19, 2024

Hi there, please see the debug info below


Yazi
    Version: 0.3.1 (4112bf4 2024-08-15)
    Debug  : false
    OS     : windows-x86_64 (windows)

Ya
    Version: 0.3.1 (4112bf4 2024-08-15)

Emulator
    Emulator.via_env: ("", "")
    Emulator.via_csi: Ok(Unknown([]))
    Emulator.detect : Unknown([])

Adapter
    Adapter.matches: Chafa

Desktop
    XDG_SESSION_TYPE: None
    WAYLAND_DISPLAY : None
    DISPLAY         : None

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL              : None
    EDITOR             : None
    VISUAL             : None
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : Some("c:\\tools\\Scripts\\yazi_config")

Text Opener
    default: Some(Opener { run: "nvim \"%*\"", block: true, orphan: false, desc: "nvim", for_: None, spread: true })
    block  : Some(Opener { run: "nvim \"%*\"", block: true, orphan: false, desc: "nvim", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : program not found
    ZELLIJ_SESSION_NAME: None
    Zellij version     : program not found

Dependencies
    file             : 5.45
    ueberzugpp       : program not found
    ffmpegthumbnailer: program not found
    magick           : program not found
    fzf              : 0.54.3
    fd               : 10.1.0
    rg               : 14.1.0
    chafa            : program not found
    zoxide           : 0.9.4
    7z               : program not found
    7zz              : program not found
    jq               : program not found


--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "C:\\Users\\abcdef\\AppData\\Roaming\\yazi\\state" directory.


@github-actions github-actions bot removed the waiting on op Waiting for more information from the original poster label Aug 19, 2024
@sxyazi
Copy link
Owner

sxyazi commented Aug 19, 2024

Emulator.via_csi: Ok(Unknown([]))

It seems that MS Terminal isn't returning any response that indicating supports Sixel. Can you confirm if your terminal supports Sixel? It looks like the PR you linked was merged very recently.

@sxyazi sxyazi added the waiting on op Waiting for more information from the original poster label Aug 19, 2024
@echaya
Copy link
Author

echaya commented Aug 19, 2024

Seems you are right. I got overexcited =)

Can we keep this open. I'll revert back once it is available in the preview version. Thanks

@github-actions github-actions bot removed the waiting on op Waiting for more information from the original poster label Aug 19, 2024
@sxyazi
Copy link
Owner

sxyazi commented Aug 19, 2024

I don't think there's any point in keeping it open since it's a feature that's already supported.

As soon as MS Terminal supports Sixel and returns the correct response, Yazi will be able to recognize and use it. In other words, there's no additional work needed on Yazi's side.

@echaya
Copy link
Author

echaya commented Aug 19, 2024

Nice - thanks for explaining

@echaya echaya closed this as completed Aug 19, 2024
@sxyazi
Copy link
Owner

sxyazi commented Aug 19, 2024

Of course if MS Terminal supports Sixel but you find it doesn't work with Yazi, feel free to create a new bug report anytime :)

@sxyazi sxyazi closed this as not planned Won't fix, can't repro, duplicate, stale Aug 19, 2024
Copy link

I'm going to lock this issue because it has been closed for 30 days. ⏳
This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants