search

amonks / run

Go run some tasks!
https://amonks.github.io/run/
Other
19 stars 4 forks source link

tui feedback in a light terminal #130

Closed jellevandenhooff closed 5 months ago

jellevandenhooff commented 5 months ago

An image says more than a thousand words:

image

That's a lot of yellow!

Also, I wish the @watch didn't blink. It's nice to know that run is still working, but the blinking is distracting.

cgsdev0 commented 5 months ago

@jellevandenhooff which terminal is this? this looks very busted

jellevandenhooff commented 5 months ago

stock terminal.app in macos 14.4.1 running zsh

amonks commented 5 months ago

what's your $TERM?

jellevandenhooff commented 5 months ago 
➜  ~ echo $TERM
xterm-256color
amonks commented 5 months ago

I can repro! thanks.

I imagine we're using some ansi sequence that Terminal.app doesn't support properly.

In the meantime, TERM=screen run ... (which disables some features) might look marginally better, depending on your color scheme.

amonks commented 5 months ago

@jellevandenhooff here's a branch that disables color entirely, in case that's helpful before there's a real fix: amonks/no-color

(this was a byproduct of trying to find the offensive color, not something I made specially for you to be helpful; don't, like, feel bad if you don't use it)

amonks commented 5 months ago

fixed in v1.0.0-beta.30

thanks again @jellevandenhooff for the report

jellevandenhooff commented 4 months ago

oh I love the colors now that they work! thanks.

when installing, the new version wasn't picked up by go install ...@latest and I had to manually put the version in. suspect a go module cache issue?

amonks commented 4 months ago

I get 30 when I install @latest now, so perhaps installing it explicitly busted the cache.

later...

yeah, based on the docs here https://index.golang.org/#faq-new-version, it looks like updates take an unspecified amount of time to show up, but you can force them by go getting the new version explicitly. I'll add this to the release script (#137).

Thanks for reporting this!