p5.js-web-editor icon indicating copy to clipboard operation
p5.js-web-editor copied to clipboard

Show a proper loading state instead of “No sketches”

Open SableRaf opened this issue 4 months ago • 9 comments

Increasing Access

Reduce confusion and anxiety when loading a large collection of sketches.

Feature enhancement details

Right now the editor shows “No sketches” while data is still loading. For large collections this can last several seconds or longer, making it look like all sketches are gone 😱

Even though I've been aware of the issue for a while, my heart always skips a beat when this happens 😅

Proposed fix

  • Only show “No sketches” when the API confirms zero results
  • Use a loading state (spinner or skeletons, aria-live="polite") while fetching
  • Add an error state with retry if loading fails

Screenshot

Image

SableRaf avatar Aug 27 '25 12:08 SableRaf

Thanks for opening this! I can see this definitely giving quite a surprise 😂

Looking into SketchList.jsx, it seem like we do have something in place to have the Loader appear instead of an empty table. I'm assuming there could probably be some adjustments here that would help resolve this!

raclim avatar Aug 27 '25 12:08 raclim

Hi @raclim , I'd like to work on this issue! I’ll implement a proper loading state, delay the “No sketches” message until the API confirms it, and add basic error handling with retry. Let me know if that works, and feel free to assign it to me.

Nitin2332 avatar Sep 03 '25 03:09 Nitin2332

Hi @raclim ,I would like to work on this issue and can you tell me how can i get to this page .

shaswata-26 avatar Sep 04 '25 11:09 shaswata-26

hii @raclim, I’d like to work on this issue, assigning myself if that’s fine.

Arav-Menon avatar Sep 06 '25 23:09 Arav-Menon

hii @raclim

can i solve this issue can you assign this issue to me ??

princekamariya avatar Sep 07 '25 12:09 princekamariya

hii @raclim

I would love to work on this issue,can you assign this to me?

Anshv784 avatar Sep 08 '25 06:09 Anshv784

Thanks so much for your interest in this issue! Since @Nitin2332 was the first to share an outline of the solution, I’ll go ahead and assign it to them. Really appreciate everyone’s enthusiasm!

raclim avatar Sep 08 '25 12:09 raclim

@Nitin2332 hi, are you working on this issue? If not I would like to solve this.

Paulie-Aditya avatar Sep 12 '25 08:09 Paulie-Aditya

@Paulie-Aditya Hi! Yes, I’m currently working on this issue.

Nitin2332 avatar Sep 12 '25 08:09 Nitin2332