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

Container table age column will not display Starting... Stopping.. #7734

Closed
axel7083 opened this issue Jun 19, 2024 · 4 comments
Closed

Container table age column will not display Starting... Stopping.. #7734

axel7083 opened this issue Jun 19, 2024 · 4 comments
Labels
area/ui kind/enhancement ✨ Issue for requesting an improvement

Comments

@axel7083
Copy link
Contributor

Is your enhancement related to a problem? Please describe

As requested by @cdrage in #7725 (comment) with this PR merged, the age column will only show duration (as it should).

Meaning we will stop showing the Starting... and Stopping... text in this column

This is not really a problem has we already show this in another place

image

Describe the solution you'd like

#7725 merged

Describe alternatives you've considered

No response

Additional context

No response

@cdrage
Copy link
Contributor

cdrage commented Jun 19, 2024

@ekidneyrh

@deboer-tim
Copy link
Collaborator

Thanks for opening @axel7083. As per my comment in 7725, the Starting... and Stopping... in this column was originally a UX suggestion for both Containers and Pods pages to make the state change more obvious (you clicked a button and something is now happening). In the Pods case we don't have the text in the Name column, but in my opinion both are somewhat random (why is state in Name column or state change in Age, and when we also have a Status column?), and worse because we don't do them in any other pages.

IMHO we should have a minor UX decision on whether we need to show the state text anywhere other than the Status column/tooltip, and ditto for state changes, and then make things consistent on all pages. But personally I'd be fine if we just removed from Pods as well and deleted StateChange.

@ekidneyrh
Copy link
Contributor

I think it's good to have that feedback that something is happening, so along as something changes somewhere it's fine by me to remove the starting and stopping from the age

@axel7083
Copy link
Contributor Author

axel7083 commented Jul 2, 2024

Closing as confirmed we do not need it anymore

@axel7083 axel7083 closed this as not planned Won't fix, can't repro, duplicate, stale Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui kind/enhancement ✨ Issue for requesting an improvement
Projects
Status: ✔️ Done
Development

No branches or pull requests

6 participants