-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Show workspace details in usage list view #11640
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
Comments
Looks great, some thoughts I had:
|
Some thoughts on the feedback points from bottom to top, while keeping in mind this is still work-in-progress:
Good point! 🤝
Agree, we've already changed this to
Yes! See #11526 (comment).
This was inspired from a past discussion in #10328 (comment) with the intent to start exposing users to the concept of session which could be useful also on the workspaces list later. However, I agree we could skip that for now and re-introduce later. 🤝
This is related to whether we surface running workspace or prebuilds in this list, no? So far, the consensus I've seen is to include running workspaces (see. relevant discussion (internal)) but I'm not sure about the current implementation. If we are going to include running workspaces, we'd have to make sure the STOPPED time remains blank or includes an indicator this is still running, etc.
Good point! This was probably left outside while tackling data structure which is different for workspaces and prebuilds, like the workspace ID v. prebuild ID and workspace owner vs. missing prebuild triggerer (see relevant note (internal)).
Thanks @laushinka for opening this and @svenefftinge for the feedback! |
Posting a new iteration of the workspaces details, including font color, weight, and alignment on the table in the scope of #11527. Changes from the last iteration include the following:
See design specs. |
Adding below the updated design specs for the usage list after this morning's discussion and pairing session with @laushinka and @jldec. 🍐
See design specs. |
Uh oh!
There was an error while loading. Please reload this page.
@jldec updated Aug 9
Workspace details in a dropdown or inline in each row
Let's increase the page size to 50 for now (#12122)
Must haves
**Nice to have (ok skip for now) **
From Figma
NOTE: Other design changes are discussed in separate issue #11953
The text was updated successfully, but these errors were encountered: