abra app ls improvements #691

Closed
opened 2025-10-03 20:59:07 +00:00 by decentral1se · 3 comments
Owner
  • Don't explode if the catalogue is missing
  • Turn off logging (like we do with the deploy TUI)
  • Pipe warning/errors into the footer (e.g. "server missing context")
  • Multiling/height is still broken, e.g. --status for an app that has 10 upgrades
  • The "querying status" horizontal layout is too wide sometimes and doesn't resize
* [ ] Don't explode if the catalogue is missing * [ ] Turn off logging (like we do with the deploy TUI) * [ ] Pipe warning/errors into the footer (e.g. "server missing context") * [ ] Multiling/height is still broken, e.g. `--status` for an app that has 10 upgrades * [ ] The "querying status" horizontal layout is too wide sometimes and doesn't resize
decentral1se added this to the Abra "next" project 2025-10-03 20:59:29 +00:00
decentral1se moved this to In Progress in Abra "next" on 2025-10-03 20:59:51 +00:00
decentral1se added the
bug
label 2025-10-03 21:00:35 +00:00
Author
Owner

OK, the table implementation we're using currently needs adjusting. It can't fit things into the height properly. I'd rather not implement this. I think https://github.com/charmbracelet/bubbles?tab=readme-ov-file#list fits the requirements and supports fuzzy finding and laying out meta information. I think a table is ideal but the standard tables don't really fit for layouts on terminal screens with so much information. I'm trying to not end up implementing a full UI and just re-use existing parts. Will check out the list layout soon.

OK, the table implementation we're using currently needs adjusting. It can't fit things into the height properly. I'd rather not implement this. I think https://github.com/charmbracelet/bubbles?tab=readme-ov-file#list fits the requirements and supports fuzzy finding and laying out meta information. I think a table is ideal but the standard tables don't really fit for layouts on terminal screens with so much information. I'm trying to not end up implementing a full UI and just re-use existing parts. Will check out the list layout soon.
Author
Owner

Additionally, we'll need to adjust the tests which use app ls to get a listing and evaluate output. These won't have output anymore. I imagine -m can work for the tests. The nightly CI is broken currently because of this (I think).

Additionally, we'll need to adjust the tests which use `app ls` to get a listing and evaluate output. These won't have output anymore. I imagine `-m` can work for the tests. The nightly CI is broken currently because of this (I think).
Author
Owner

I think we need a UI designer to make some progress on this as @kawaiipunk mentioned. I am stuck. I see other bugs that could be getting fixed in the meantime, so I'm gonna revert / work around this for now. I think we definitely need to improve abra app ls but I'm just not sure what is the best approach.

I think we need a UI designer to make some progress on this as @kawaiipunk mentioned. I am stuck. I see other bugs that could be getting fixed in the meantime, so I'm gonna revert / work around this for now. I think we definitely need to improve `abra app ls` but I'm just not sure what is the best approach.
decentral1se moved this to Done in Abra "next" on 2025-10-17 17:30:15 +00:00
decentral1se removed this from the Abra "next" project 2025-10-17 17:31:14 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: toolshed/abra#691
No description provided.