Skip to content

preload and async setup loading animation #48

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

Open
quinton-ashley opened this issue Jun 20, 2024 · 3 comments
Open

preload and async setup loading animation #48

quinton-ashley opened this issue Jun 20, 2024 · 3 comments
Labels
deferred enhancement New feature or request

Comments

@quinton-ashley
Copy link
Collaborator

quinton-ashley commented Jun 20, 2024

@SableRaf I like this idea!

processing/p5.js#6795

I did something similar with p5play with the "made with p5play" screen but it fills the window, assuming nearly all usage of p5play is in global mode.

For q5, I'm thinking of doing a loading animation of the q5 logo spinning in css right above the center of the canvas.

In response to the opt-in idea, I think that wouldn't work because developers loading assets locally may never experience long loading times, so they wouldn't see the message suggesting to enable it. Also my assumption is a loading animation would always be preferable to "loading" text.

@quinton-ashley quinton-ashley added the enhancement New feature or request label Jun 20, 2024
@quinton-ashley quinton-ashley self-assigned this Jun 20, 2024
@RandomGamingDev
Copy link

I agree with it being opt-out, but I think you should still be able to customize the loading animation. What are your thoughts on that?

@quinton-ashley quinton-ashley changed the title Loading animation preload and async setup loading animation Jun 20, 2024
@quinton-ashley
Copy link
Collaborator Author

quinton-ashley commented Jun 20, 2024

yeah for sure, I'm thinking users could opt out like this:

Q5.preloadAnimation = false

Would need to be set before preload is run

@quinton-ashley quinton-ashley removed their assignment Apr 9, 2025
@quinton-ashley quinton-ashley moved this to Maybe in q5 Apr 10, 2025
@quinton-ashley
Copy link
Collaborator Author

Kit approved this so I think it's best to wait until this is implemented in p5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deferred enhancement New feature or request
Projects
Status: Maybe
Development

No branches or pull requests

2 participants