-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[dashboard] introduce folded inactive ws section #10676
Conversation
9e9bb32
to
2ab49f3
Compare
started the job as gitpod-build-sefftinge-make-it-easier-to-delete-4233.19 because the annotations in the pull request description changed |
@@ -37,6 +37,9 @@ export default function Modal(props: { | |||
.catch(console.error); | |||
}; | |||
const handler = (evt: KeyboardEvent) => { | |||
if (!props.visible) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UX LGTM. ✔️
@svenefftinge This probably resolves also #10662, right?
started the job as gitpod-build-sefftinge-make-it-easier-to-delete-4233.20 because the annotations in the pull request description changed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome!
Thanks for fixing it in a holistic way!
Works fine for me.
This reapplies the changes from the previously reverted PR #10450 and fixes
Description
Many users are manually deleting workspaces, keeping their workspaces list clean.
Users should feel less urge to manually clean up and cleaning up should be simpler.
This PR, makes the inactive workspaces section on the /workspaces list collapsable (collapsed by default) to give a cleaner impression. When expanded there is a button to delete them.
Related Issue(s)
Fixes #10662
Fixes #10665
Fixes #1053
Fixes #4233
How to test
Use the preview env and start a few workspaces, stop them. Check the workspaces list.
Release Notes
Documentation