search

zed-industries / zed

Code at the speed of thought – Zed is a high-performance, multiplayer code editor from the creators of Atom and Tree-sitter.
https://zed.dev
Other
49.44k stars 3.01k forks source link

Some console output text is invisible #7281

Open hubertpompecki opened 9 months ago

hubertpompecki commented 9 months ago

Check for existing issues

Describe the bug / provide steps to reproduce it

I keep running into an issue with some text (paths to failing tests in the output of mix test when working with Elixir) not being visible in the Zed terminal. This happens in all colour themes and across multiple Elixir versions. image I need to either hover over or highlight the text to see it: image I thought this might be somehow related to my zsh config but the issue persists when setting the terminal to sh with:

    "shell": {
      "program": "sh"
    }

image

When running the same test in iTerm or terminal the path is displayed fine. image

Environment

Zed: v0.120.5 (Zed) OS: macOS 14.0.0 Memory: 16 GiB Architecture: x86_64

If applicable, add mockups / screenshots to help explain present your vision of the feature

No response

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.

No response

bluesbettle commented 9 months ago

I can reproduce on: Zed: v.0.120.6 OS: macOS 14.1.2 (Sonoma) Architecture (ARM M1) Shell:

bash --version
GNU bash, version 5.2.15(1)-release
hubertpompecki commented 7 months ago

For anyone running into this, whilst all default colour schemes are affected, I was able to get the missing line to show by using any one of catppuccin themes.

github-actions[bot] commented 1 month ago

Hi there! 👋 We're working to clean up our issue tracker by closing older issues that might not be relevant anymore. Are you able to reproduce this issue in the latest version of Zed? If so, please let us know by commenting on this issue and we will keep it open; otherwise, we'll close it in a week. Feel free to open a new issue if you're seeing this message after the issue has been closed. Thanks for your help!

hubertpompecki commented 1 month ago

Commenting because this still happens with all built-in themes.

notpeter commented 2 days ago

I found reference to this upstream: Mix test hides console output text with black on black colorization?

Its suggests that this text is ansi black + ansi bright; and so perhaps themes with a black terminal background should render this as a color other than just plain black.