Git Product home page Git Product logo

Comments (15)

Canop avatar Canop commented on May 25, 2024 3

The fix has been released with version 1.36.1 @CarterLi @zeenix @Gelio

from broot.

Gelio avatar Gelio commented on May 25, 2024 1

You are right, this problem does not occur on 1.32.0

Kapture.2024-03-10.at.12.00.19.mp4

For reference, I am using kitty 0.32.2

from broot.

Gelio avatar Gelio commented on May 25, 2024 1

I understand you wanted me to run cargo run --example kitty from that repo.

Here is the output in kitty in tmux:

image

Here is the output in kitty outside of tmux:

image

The extra codes in my prompt were inserted automatically. The screenshots contain the state right after the command finished

from broot.

zeenix avatar zeenix commented on May 25, 2024 1

I checked out that commit and it looks like the problem is fixed for me on kitty

Nice! Thanks so much for testing.

Interestingly, the macOS Terminal app still has the extra characters:

That's just the Terminal app being silly AFAICT. If you revert the commit to before my recent changes (git checkout abfb5aae9e7539cb9cd3c8b3fcc98a7dfa5e6276) and run the example, you'll still see those. I'd just say do what everyone does on Mac: stick to iTerm2. :)

@Gelio just out of curiosity, was broot working fine on Mac before recently?

I switched to Mac recently and I can't tell, as I haven't used broot a lot recently :/ sorry

No worries. I can verify that at least kitty worked fine before my recent changes. My own app (gimoji) didn't work correctly though when launched from a git hook. In any case, I think we've now a fix for all that is fixable. :)

from broot.

CarterLi avatar CarterLi commented on May 25, 2024 1

Same issue for iTerm in macOS

from broot.

Gelio avatar Gelio commented on May 25, 2024 1

I just upgraded and the problem no longer exists for me on Kitty (both in and outside of tmux)

Thanks @Canop and @zeenix for delivering a quick fix 🚀

from broot.

Canop avatar Canop commented on May 25, 2024

This is related to the test of terminal colors.

Assuming you're using version 1.36 of broot, can you please see if it happens for you with version 1.32 ?

from broot.

Canop avatar Canop commented on May 25, 2024

@Gelio I don't know if you're able to compile things for rust.

If so, I would appreciate if you could test the example of https://github.com/Canop/xterm-query/ which is executed with cargo run --example kitty

from broot.

Canop avatar Canop commented on May 25, 2024

What's your terminal and OS exactly ?

from broot.

zeenix avatar zeenix commented on May 25, 2024

Strange. Ironically my commit in xterm-query introducing this issue is exactly the solution to the symptoms here (at least on Linux) that I reported in Canop/terminal-light#4.

from broot.

Gelio avatar Gelio commented on May 25, 2024

What's your terminal and OS exactly ?

Terminal: kitty 0.32.2

OS: macOS Sonoma 14.2.1

from broot.

Gelio avatar Gelio commented on May 25, 2024

Here is the output from macOS default terminal app:

image image

from broot.

zeenix avatar zeenix commented on May 25, 2024

@Gelio Thanks for providing the example run. I was about to reproduce this issue. I think Canop/xterm-query#4 fixes this since I'm not able to reproduce the issue when using the branch of that PR.

from broot.

zeenix avatar zeenix commented on May 25, 2024

@Gelio just out of curiosity, was broot working fine on Mac before recently?

from broot.

Gelio avatar Gelio commented on May 25, 2024

@Gelio Thanks for providing the example run. I was about to reproduce this issue. I think Canop/xterm-query#4 fixes this since I'm not able to reproduce the issue when using the branch of that PR.

I checked out that commit and it looks like the problem is fixed for me on kitty

Kitty in tmux:

image

Kitty outside of tmux (says image protocol is supported, as expected):

image

Interestingly, the macOS Terminal app still has the extra characters:

image

@Gelio just out of curiosity, was broot working fine on Mac before recently?

I switched to Mac recently and I can't tell, as I haven't used broot a lot recently :/ sorry

from broot.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.