Convert abra app ps to vertical layout #487

New Issue

decentral1se · 2025-01-16T11:08:38Z

decentral1se commented

2025-01-16 11:08:38 +00:00

This has been discussed on #coop-cloud-tech:autonomic.zone.

There is no ... to show anything is truncated.

A vertical layout may be more ideal. @fauno shared the \G trick:

https://pento.net/2009/02/27/the-g-modifier-in-the-mysql-command-line-client/

/cc @kawaiipunk @aadil

<img width="484" alt="image" src="attachments/e2012d57-cdb5-4a2d-80a2-1c7b06c057e8"> This has been discussed on `#coop-cloud-tech:autonomic.zone`. There is no `...` to show anything is truncated. A vertical layout may be more ideal. @fauno shared the `\G` trick: https://pento.net/2009/02/27/the-g-modifier-in-the-mysql-command-line-client/ /cc @kawaiipunk @aadil

image.png

32 KiB

decentral1se added the

enhancement

design

labels 2025-01-16 11:08:49 +00:00

decentral1se commented

2025-01-16 11:18:37 +00:00

My preference: just use the vertical layout (like the deploy screen). Don't bother with an option between table or vertical. Putting ... into the table layout might involve fucking around with the internals of the renderer so I'm not keen.

My preference: just use the vertical layout (like the deploy screen). Don't bother with an option between table or vertical. Putting `...` into the table layout might involve fucking around with the internals of the renderer so I'm not keen.

👍 1

simon commented

2025-01-22 11:07:11 +00:00

As we're talking layout here anyways, I'd copy over my thoughts on it from Matrix:

I'd vote for putting the Status column next to the Image column, maybe even between Service and Image as those are values I look often at in exactly this context. With many services for an app, it gets difficult to not get lost in the lines
Also the information densitity is a lot higher in the previous layout. I like that you put the Version and Chaos in the table, but maybe we could give it its own row, as its the same across the whole app?

see previous and current layout attached.

Edit: Also made a quick paint draft to visualize what I imagine

As we're talking layout here anyways, I'd copy over my thoughts on it from Matrix: I'd vote for putting the `Status `column next to the `Image `column, maybe even between `Service `and `Image `as those are values I look often at in exactly this context. With many services for an app, it gets difficult to not get lost in the lines Also the information densitity is a lot higher in the previous layout. I like that you put the `Version `and `Chaos `in the table, but maybe we could give it its own row, as its the same across the whole app? see previous and current layout attached. Edit: Also made a quick paint draft to visualize what I imagine

grafik(1).png

97 KiB

grafik(2).png

19 KiB

👀 1

decentral1se commented

2025-02-05 13:56:29 +00:00

tysm @simon i'll get back to you on this. really appreciate the feedback. i think it'd be good to get in your adjustments in the release candidate. (and I would still postpone the vertical layout refactor)

decentral1se added this to the abra v0.10.0 project 2025-02-05 13:56:41 +00:00

decentral1se added the

release-candidate

label 2025-02-12 14:56:50 +00:00

decentral1se changed title from ~~`abra app ps` truncated "silently"~~ to Convert `abra app ps` to vertical layout

2025-03-13 09:52:11 +00:00

decentral1se removed the

release-candidate

label 2025-03-13 09:52:19 +00:00

decentral1se removed this from the abra v0.10.0 project 2025-03-13 09:52:23 +00:00

decentral1se referenced this issue

2025-03-13 09:53:26 +00:00

`abra app ps` layout tweaks #513

decentral1se commented

2025-03-13 09:53:51 +00:00

Following #487 (comment)

#513

Following https://git.coopcloud.tech/toolshed/abra/issues/487#issuecomment-22739 https://git.coopcloud.tech/toolshed/abra/issues/513

decentral1se referenced this issue

2025-03-16 11:43:33 +00:00

refactor!: status between service/image #516

decentral1se commented

2025-03-16 11:46:32 +00:00

I'm actually gonna close this off also because in the upcoming RC we have:

wrapping in the cells, so you don't have truncation at all
width based on terminal size, with preference for 79 chars
"status" shown between service/image

It is possible to show truncation via #513 (comment) but I chose for wrapping. We can change that if people think it is better to wrap.

The vertical layout can come later if people still think the overview is not ideal. Or we need more information.

See also #513.

Thanks for the feedback everyone!

I'm actually gonna close this off also because in the upcoming RC we have: * wrapping in the cells, so you don't have truncation at all * width based on terminal size, with preference for 79 chars * "status" shown between service/image It is possible to show truncation via https://git.coopcloud.tech/toolshed/abra/issues/513#issuecomment-23054 but I chose for wrapping. We can change that if people think it is better to wrap. The vertical layout can come later if people still think the overview is not ideal. Or we need more information. See also https://git.coopcloud.tech/toolshed/abra/issues/513. Thanks for the feedback everyone!

decentral1se closed this issue

2025-03-16 11:46:32 +00:00

decentral1se added this to the abra v0.10.0 project 2025-03-16 11:46:38 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-14 22:00:59 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-14 22:01:01 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-16 05:16:08 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-16 05:16:10 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-19 07:28:29 +00:00

decentral1se moved this to Done in abra v0.10.0 on 2025-04-21 17:48:16 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: toolshed/abra#487

Convert abra app ps to vertical layout #487

Convert `abra app ps` to vertical layout #487