Skip to content
This repository has been archived by the owner on Mar 21, 2023. It is now read-only.

Design Explorations: Error States #11

Open
melchoyce opened this issue Jul 10, 2019 · 17 comments
Open

Design Explorations: Error States #11

melchoyce opened this issue Jul 10, 2019 · 17 comments
Assignees

Comments

@melchoyce
Copy link
Collaborator

melchoyce commented Jul 10, 2019

We need to figure out all the potential errors and where there can be points of failure:

  • Block can’t install (your internet craps out, etc.)
  • What happens if you don’t have permissions
  • What else?

Related: #8

@melchoyce melchoyce self-assigned this Jul 10, 2019
@melchoyce
Copy link
Collaborator Author

If we can't preview the block for some reason, like a WordPress.org outage:

image

@ck3lee do you know if we have existing error strings for something like this? If not, we'll want to write one.

If the installation fails (but the block can preview):

image


Is there any other point in this flow that might produce an error?

@melchoyce
Copy link
Collaborator Author

melchoyce commented Jul 10, 2019

Permissions

If you don't have administrative permissions (or any custom permission that allows installation of new plugins), let's show the existing message for search results:

@boemedia
Copy link

In the last case, is it possible to show the blocks but greyed out, with a message saying:
"Your current role doesn't allow installing new blocks. Contact your website's administrator."

@melchoyce
Copy link
Collaborator Author

Maybe like:

image

@karmatosed
Copy link
Member

I like that as an option over having greyed out. I feel if we showed greyed out blocks it could taunt the user.

@ck-lee
Copy link

ck-lee commented Jul 11, 2019

do you know if we have existing error strings for something like this?

No, I couldn't find any existing error strings. I only found a couple that are generic and technical like
The response is not a valid JSON response.. So, we might need to write them.

@ck-lee
Copy link

ck-lee commented Jul 11, 2019

LGTM! I think the two scenarios: "can't preview the block" and "installation fails" should capture all the unexpected failure points. I will drop a comment here if I can think of anything else.

@ck-lee
Copy link

ck-lee commented Jul 15, 2019

@melchoyce, I'm finding it very tricky to implement the actions in the notice bar. I am finding out more from the Gutenberg dev team now.

In case, it is a mission to add the actions like "Retry" or "Remove block", is it OK to remove them?

@melchoyce
Copy link
Collaborator Author

As long as we can add them to a later roadmap and follow up, I think it's okay to drop from the v1.

@ck-lee
Copy link

ck-lee commented Jul 17, 2019

Thank you @melchoyce. Looks like we can do a "Retry". 😄

By default all "non-url actions" in the notice are rendered as buttons instead of links. I think it might be semantically more appropriate for a11y. But happy to re-style that as a link too.

Still a WIP. But here is what is looks like:
screencast 2019-07-18 10-55-16

@ck-lee
Copy link

ck-lee commented Jul 17, 2019

And for user without permission to install a block.

This is how it looks like:

screencast 2019-07-18 10-57-12

@karmatosed
Copy link
Member

@melchoyce I wonder if the 'no blocks' could be an opportunity for a cute little graphic? Like a sad block. It's just an idea.

@karmatosed
Copy link
Member

Just checking back on this, could it be in v1 to explore a little visual variation beyond text?

@melchoyce
Copy link
Collaborator Author

Should we explore this in Gutenberg core? We're hooking in to the core "no results" pattern.

@karmatosed
Copy link
Member

Sure, I will make issue for this also.

@melchoyce
Copy link
Collaborator Author

Thanks, feel free to assign it to me.

@karmatosed
Copy link
Member

karmatosed commented Aug 21, 2019

@melchoyce WordPress/gutenberg#17122 is where started that issue.

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

No branches or pull requests

4 participants