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

[Feature] Prevent displaying broken image while image is processing #514

Open
colbyfayock opened this issue Sep 8, 2024 · 11 comments
Open
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@colbyfayock
Copy link
Collaborator

Feature Request

Is your feature request related to a problem? Please describe.

For some transformations like background removal and AI transformations, the first request may recieve a 423 if it needs to be processed.

The CldImage component currently polls the image until its finished and refreshes the DOM, but in the meantime, the image is displayed as broken on the page.

How can we prevent the image appearing visually broken?

Can somewhat try to replicate the way the underlaying Next Image component is handling placeholders potentially

@sahil9001
Copy link
Contributor

Hey @colbyfayock , Can you please assign this to me?

@colbyfayock
Copy link
Collaborator Author

all yours! let me know if you have any questions or if anything is unclear

@sahil9001
Copy link
Contributor

Hey @colbyfayock, how do I get the removed background image? When I used the dev setup they just simply won't load, I needed it to load as I am using original image blurring so wanted to see the blue over original image

@sahil9001
Copy link
Contributor

@colbyfayock Also, how do we want to proceed with this? Do we want to show the spinner or the blurred image?

@colbyfayock
Copy link
Collaborator Author

@sahil9001 you need to enable the Backhround Removal Add-On within your Cloudinary account

https://cloudinary.com/documentation/cloudinary_ai_background_removal_addon

I'm open to suggestions, I was thinking of something a bit unopinionated where not everyone may want a spinner, so perhaps a blurred image would be nice, especially because you can use the original image without the background removed to achieve that

@colbyfayock
Copy link
Collaborator Author

@sahil9001
Copy link
Contributor

@sahil9001 you need to enable the Backhround Removal Add-On within your Cloudinary account

https://cloudinary.com/documentation/cloudinary_ai_background_removal_addon

I'm open to suggestions, I was thinking of something a bit unopinionated where not everyone may want a spinner, so perhaps a blurred image would be nice, especially because you can use the original image without the background removed to achieve that

That would also mean that I have to buy subscription for background removal? Because on the UI it says to buy subscription in order to use it.

@colbyfayock
Copy link
Collaborator Author

it comes with 15 free removals under a free tier

@sahil9001
Copy link
Contributor

I'm open to suggestions, I was thinking of something a bit unopinionated where not everyone may want a spinner, so perhaps a blurred image would be nice, especially because you can use the original image without the background removed to achieve that

I agree with this, thanks. Raising a PR

@colbyfayock
Copy link
Collaborator Author

@sahil9001 any updates?

@sahil9001
Copy link
Contributor

Sorry man, got caught up with few things here, I will do this by tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants