Change chaos version marker for size constrained layouts? #447

New Issue

decentral1se · 2023-04-16T11:26:42Z

decentral1se commented

2023-04-16 11:26:42 +00:00

343b2bfb91/pkg/recipe/recipe.go (L420)

	if !isClean {
		version = fmt.Sprintf("%s + unstaged changes", version)
	}

@moritz could we change this to:

	if !isClean {
		version = fmt.Sprintf("%s*", version)
	}

Where * signals "unstaged changes". Or some other shorter marker?

This is because we are already quite constrained in the screen real estate we have for laying out versions and such on abra app ls and in my experiments with a TUI dashboard.

So, this would save us some space for the table layouts, e.g.

https://git.coopcloud.tech/coop-cloud/abra/src/commit/343b2bfb91b583e75b4c963194208f2ca2a41e6e/pkg/recipe/recipe.go#L420 ```go if !isClean { version = fmt.Sprintf("%s + unstaged changes", version) } ``` @moritz could we change this to: ```go if !isClean { version = fmt.Sprintf("%s*", version) } ``` Where `*` signals "unstaged changes". Or some other shorter marker? This is because we are already quite constrained in the screen real estate we have for laying out versions and such on `abra app ls` and in my experiments with a TUI dashboard. So, this would save us some space for the table layouts, e.g. ![image](/attachments/6c936849-a9ea-4fe3-9192-1543d0fef623) ![image](/attachments/1fe47dfd-1a76-4a25-8821-2ecd042730e6)

image.png

22 KiB

image.png

44 KiB

decentral1se added the

design

proposal

labels 2023-04-16 11:26:42 +00:00

moritz commented

2023-04-17 14:42:00 +00:00

I think this is a good idea. Maybe an extra line at the bottom *: ustaged changes would be helpful.

I think this is a good idea. Maybe an extra line at the bottom `*: ustaged changes` would be helpful.