search

wtfutil / wtf

The personal information dashboard for your terminal
http://wtfutil.com
Mozilla Public License 2.0
15.89k stars 805 forks source link

cmdrunner doesn't show proper color (`term` setting not used?) #1267

Closed albert-ying closed 1 year ago

albert-ying commented 2 years ago

What's broken?

When I run task burndown.daily in terminal

image

In wtfutil

    taskburndown:
      args: ["burndown.daily"]
      cmd: "task"
      enabled: true
      position:
        top: 0
        left: 3
        height: 2
        width: 2
      refreshInterval: 30
      type: cmdrunner
image
albert-ying commented 2 years ago

The problem persists even after I add term: "xterm-256color" to the config yml --

Any idea what is happening here?

albert-ying commented 2 years ago

Seems that term setting is not respected. I didn't get any error even I set it to some random string

term Sets a custom value for the terminal type this app runs in. Leave this entry out of the config if you simply want to use your terminal's default setting. Note: If an invalid value is provided for this setting, the app will crash with a terminal entry not found error.

senorprogrammer commented 2 years ago

Possibly related to this issue https://github.com/wtfutil/wtf/issues/577, specifically this comment? https://github.com/wtfutil/wtf/issues/577#issuecomment-526835771

Seanstoppable commented 2 years ago

If you set pty to true, it definately tries to do something else. Unfortunately, in my case, that something else is printing out Terminal window too small to draw a graph.