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

Improve UX on the dashboard page when a workspace is stopped / idled #21540

Closed
ibuziuk opened this issue Jul 8, 2022 · 18 comments
Closed

Improve UX on the dashboard page when a workspace is stopped / idled #21540

ibuziuk opened this issue Jul 8, 2022 · 18 comments
Assignees
Labels
area/ux Issues related to User Experience Design kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@ibuziuk
Copy link
Member

ibuziuk commented Jul 8, 2022

Is your task related to a problem? Please describe

The original page has been implemented as part of #20599

image

Describe the solution you'd like

Page that follows UX patterns - Look and feel suggestions are most welcome
Copy/UI text wording - if you are away from you desk and step back, what should it say.
Your admin set a setting so your pods you use just stopped because stepped away

Describe alternatives you've considered

N/A

Additional context

No response

@ibuziuk ibuziuk added kind/task Internal things, technical debt, and to-do tasks to be performed. area/ux Issues related to User Experience Design sprint/next severity/P2 Has a minor but important impact to the usage or development of the system. labels Jul 8, 2022
@ibuziuk ibuziuk changed the title Improve UX on the error page when a workspace is stopped / idled Improve UX on the dashboard page when a workspace is stopped / idled Jul 8, 2022
@beaumorley
Copy link

@ibuziuk would it be possible to get a screen capture of the whole screen when this message is displayed? Thanks!

@dkwon17
Copy link
Contributor

dkwon17 commented Jul 26, 2022

Hi @beaumorley ,

This is how the whole screen looks like when the message is displayed:
image

@max-cx
Copy link

max-cx commented Jul 27, 2022

sync'd to Red Hat Jira https://issues.redhat.com/browse/CRW-3181

@dkwon17
Copy link
Contributor

dkwon17 commented Jul 27, 2022

Here is an updated screenshot:

image

Please see eclipse-che/che-dashboard#593 (comment) for more details and screenshots

@beaumorley
Copy link

beaumorley commented Jul 27, 2022

@dkwon17 Looks good! I would remove the red highlight. I think it competes with the warning icon.

We'd also like to make a recommendation for the error message and have a few questions first.

  1. Does this warning only appear due to inactivity or can it appear from other ways?
  2. What is the difference between "Restart your workspace" and "Return to dashboard"? What is the best solution for the user when they get this error?

Thanks!
cc @wolfeallison

@dkwon17
Copy link
Contributor

dkwon17 commented Jul 27, 2022

Thank you for the feedback!

  1. Does this warning only appear due to inactivity or can it appear from other ways?

After this #21516 is complete (I am currently working on this), this warning message will appear only due to inactivity. There will be other warning messages for other similar situations too.

Also, there is another issue that will be worked on: #21578, where the error message would also state the inactivity timeout. The plan is to have the error message to look something like:

Your workspace has stopped because there was no activity for 30 minutes.

How does that look/sound?

  1. What is the difference between "Restart your workspace" and "Return to dashboard"? What is the best solution for the user when they get this error?

"Restart your workspace" will launch the workspace and open the editor in the current screen:

Video
Screen.Recording.2022-07-27.at.2.18.02.PM.mov

"Return to dashboard" will direct the user to the user's workspace on the Che dashboard:

Video
Screen.Recording.2022-07-27.at.2.20.27.PM.mov

I think the best solution for the user when they get this error would be to press "Restart your workspace"

@wolfeallison
Copy link

@dkwon17 @beaumorley Awesome, thanks for answering our questions! What do you think of this: "Your workspace has stopped due to inactivity. Restart your workspace to continue using your workspace." with "Restart your workspace" as hyperlinked still and then offering "Return to dashboard" as a secondary option below it

If we know how long it takes to time out, I like the message you have planned: Your workspace has stopped because there was no activity for 30 minutes.

@dkwon17
Copy link
Contributor

dkwon17 commented Jul 27, 2022

Thank you @wolfeallison 👍 , looks good to me:
Screen Shot 2022-07-27 at 4 04 22 PM

@dkwon17
Copy link
Contributor

dkwon17 commented Jul 28, 2022

@wolfeallison @beaumorley any thoughts on adding the workspace name (ex. golang-example) into the error message?
image

@dkwon17
Copy link
Contributor

dkwon17 commented Aug 4, 2022

Also, we now have a similar error page that appears when the workspace stops due to an error:
image

I was wondering if we could also get some feedback for this page as well? In this case we also have two options: Restart your workspace or Return to Dashboard. But unlike the idling error page, for this error page I'd say there's no recommended option, it just depends on why the workspace has failed.

@nickboldt
Copy link
Contributor

Note UX questions in #21359 as well, which might impact some of the decisions here. If you have/had multiple workspaces open and one/more has idled/died, it might be worth providing a "switch to running workspace" or a list of running workspaces... but then the Dashboard link could achieve that too.

@beaumorley
Copy link

Until I saw @nickboldt comment, I did not realize that return to your dashboard was to get to other workspaces. What about instead of the last think being "Return to dashboard" it is "Return to your dashboard to switch to a running workspace." @wolfeallison do you have any thoughts on better wording?

@dkwon17
Copy link
Contributor

dkwon17 commented Aug 4, 2022

@nickboldt 's comment refers to the specific scenario where the workspace idle page is reached when there is another running workspace. Return to your dashboard to switch to a running workspace does sound like a nice enhancement to me.

But it is also possible that by the time the user returns to their computer to see the workspace idle page, the other running workspace might also have been shut down due to idling Might not be a big deal now, since the idle error page appears when the user refreshes the editor page on their own

@wolfeallison
Copy link

@nickboldt @beaumorley I like it! What about "To switch to a running workspace, return to your dashboard" and just have the return to dashboard as a link? We typically put the benefit first, which is why I recommend changing the order of the sentence

@wolfeallison
Copy link

wolfeallison commented Aug 5, 2022

for the similar error page,

What do you think of making the header "Workspace failed" and then it goes directly into the error under it?

@dkwon17
Copy link
Contributor

dkwon17 commented Aug 5, 2022

Thank you for the feedback @wolfeallison , I think those are great improvements.
Here are the updated error page screenshots with the idle timeouts (30 minutes etc.) and updated text, please let me know if there's any other feedback:

Workspace inactivity Screen Shot 2022-08-05 at 11 27 15 AM
Workspace inactivity when there is another running workspace

image

Workspace runtime limit reached (similar to inactivity timeout) Screen Shot 2022-08-05 at 11 33 55 AM
Workspace runtime limit reached when there is another running workspace

image

Workspace was not running (appears when workspace url accessed when workspace was never started)

image

Workspace stopped due to error

image

@ibuziuk
Copy link
Member Author

ibuziuk commented Aug 10, 2022

the last PR has been merged
@dkwon17 can we close the issue?

@dkwon17
Copy link
Contributor

dkwon17 commented Aug 10, 2022

All of the UX improvements are made, except for the Workspace inactivity when there is another running workspace and Workspace runtime limit reached when there is another running workspace pages, which aren't implemented yet.

We can close this issue and focus on those pages in #21623 cc @nickboldt

@dkwon17 dkwon17 closed this as completed Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ux Issues related to User Experience Design kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

6 participants