Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

table display on all commands #300

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Standing-Man
Copy link
Contributor

@Standing-Man Standing-Man commented Jan 1, 2025

fixed #304 and PR #286

fixed all commands that use table display.

current problem

image

after

image

Hi @Vad1mo and @bupd, Due to updates in the Bubble Tea framework, if the table height is set to the length of the rows, an error may occur. When the table contains only one row of data, it will display only the header. Therefore, we need to add an additional row of data to properly display the actual data.

It has been observed that any table with only one row of data encounters display issues.

@Standing-Man Standing-Man changed the title table display on artifact list table display on all commands Jan 2, 2025
@bupd
Copy link
Contributor

bupd commented Jan 9, 2025

@Standing-Man Can you check all commands such as view list and so.
Also add some screenshots for the view command

Thanks

@Standing-Man
Copy link
Contributor Author

For all view and list commands, they share the same table model for display, which is why they encounter the same issue with table presentation.
harbor project view
image

after fix this issue.
image
@bupd

@Standing-Man
Copy link
Contributor Author

ready to review. @Vad1mo @bupd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected Behaviour: Empty output in project view cmd
2 participants