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
(3P) External Media: Load More error separated from action #15895
Comments
Hey @jeryj — it seems that this happened on WordPress.com, can you confirm? |
Hi @keoshi! This isn't on WordPress.com yet, but is a new feature in Jetpack. The current Media Library on WordPress.com uses the infinite scrolling pattern. You can access this Pexels library by choosing the Free Photo Library on WordPress.com, but it also uses the infinite scrolling pattern. |
@jeryj The reason I'm asking is because from the screenshots above it seems like this is happening in Calypso rather than on a self-hosted site, even if it's a Jetpack site. If that's the case, I wonder if https://github.com/Automattic/wp-calypso would be a better place to report this issue. Some features are shared across the different code bases, so I'm unsure — cc'ing @jeherve because he'll know it. :) |
From the looks of it, this is the iFramed block editor, so that's about the Jetpack feature currently being worked on. |
@jeryj For that state to occur, users would need to lose connection between the initial API request that populates the modal and the Load more click? |
Yeah, or just have a slow connection in general and it times out. Anything that could make the request not return correctly. |
Just tested it with Chrome's "Slow 3G" and a custom 50kb/s setting and couldn't reproduce this occurring "naturally". How do you feel about classifying this one as nice-to-have for when we ran out of more important External Media tasks? |
Sounds fine. Are there any other potential issues that could happen with the Load More button? If so, it'd be good to have an error state near the button. I can't think of any off the top of my head though. |
@jancavan 👋 Are you able to take a look at this from a Design point of view? Once we have this then it will be suitable for inclusion in an estimation. |
@jeryj Out of curiosity, are there other types of error we display within this modal other than the internet connection error? |
I think so, such as if pexels doesn't initially load any results. But for those instances, there aren't photos so you're already at the top of the modal. I'm not sure of all the possible errors though. If I had to guess, most of them will happen when the modal is initially opened, rather than when clicking "load more" |
We could make the error message sticky so it's within view of the user: I originally thought of disabling the "Load more" button, but let's give the user the opportunity to retry. Upon re-attempt, error message should disappear, but will return if connection has still not yet been reestablished. A couple other minor comments:
|
Plus 1 for the sticky error message. That would be my suggestion! |
Let's close this for now, it doesn't look like it's caused any issues so far. |
Steps to reproduce the issue
What I expected
An error to show that was in view (near the load more button)
What happened instead
The text was updated successfully, but these errors were encountered: