konf-go icon indicating copy to clipboard operation
konf-go copied to clipboard

File column could be improved by either using '...' abreviation or filtering out the konfDir

Open almereyda opened this issue 2 years ago • 1 comments

Taken from README.md

almereyda avatar Aug 18 '22 13:08 almereyda

Totally agreed. Take a look at this konf set output:

$ konf set
Use the arrow keys to navigate: ↓ ↑ → ←  and / toggles search
?   Context                   | Cluster                   | File                      :
  ▸ arn:aws:eks:us-east-1:116 | arn:aws:eks:us-east-1:116 | /home/eric/.kube/konfs/st |
    arn:aws:eks:us-east-1:223 | arn:aws:eks:us-east-1:223 | /home/eric/.kube/konfs/st |
    arn:aws:eks:us-east-1:483 | arn:aws:eks:us-east-1:483 | /home/eric/.kube/konfs/st |
    arn:aws:eks:us-east-1:841 | arn:aws:eks:us-east-1:841 | /home/eric/.kube/konfs/st |
    arn:aws:eks:us-east-1:841 | arn:aws:eks:us-east-1:841 | /home/eric/.kube/konfs/st |
    dev-sandbox-usinfdevranch | dev-sandbox-usinfdevranch | /home/eric/.kube/konfs/st |
    dev-sandbox-usinfdevranch | dev-sandbox-usinfdevranch | /home/eric/.kube/konfs/st |
    dev-sandbox-usinfdevranch | dev-sandbox-usinfdevranch | /home/eric/.kube/konfs/st |
    dev-sandbox               | dev-sandbox               | /home/eric/.kube/konfs/st |
    uspttdevrafay1            | uspttdevrafay1            | /home/eric/.kube/konfs/st |

You can't tell the difference between the two rows starting with arn:aws:eks:us-east-1:841 nor the ones starting with ev-sandbox-usinfdevranch. The store directory is redundant information. Also you might as well make that column much wider since it's the last one.

sourcedelica avatar Apr 09 '23 02:04 sourcedelica