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 Request: Allow more control over placeholder #422

Open
fcisio opened this issue Aug 24, 2021 · 4 comments
Open

Feature Request: Allow more control over placeholder #422

fcisio opened this issue Aug 24, 2021 · 4 comments

Comments

@fcisio
Copy link

fcisio commented Aug 24, 2021

Hi, I love this new version of the SDK so far!
But I do which that we would have more contrôl over the image placeholders.

Examples:

  • Control some props for each mode: placeholder('pixelate', [10]), placeholder('blur', [600]), ...
  • Control the transition from placeholder to definitive image: fade, speed, delay, ... (currently the image simply blinks)
@momoip
Copy link

momoip commented Aug 24, 2021

Hi there,
We support the feature of blur, pixelate for image placeholders. Please check out:

documentation: https://cloudinary.com/documentation/react_image_manipulation#image_placeholders
tutorial: https://cloudinary.com/documentation/advanced_image_components_tutorial

Please let me know if you have further questions.
Thanks,
Mo

@fcisio
Copy link
Author

fcisio commented Aug 24, 2021

Hey, @momoip I'm aware that those are already features, this request is about extending them

Currently, those plugins only accept a string each (SDK v2), my suggestion would be to add a second argument Array which could be passed to the placeholder actions.

This array would make it possible to control the level of pixelation, blur, vectorization, ...


This could also open the door to custom placeholders by exposing the image var

placeholder('custom', [(image) => image.effect(cartoonify())])

@momoip
Copy link

momoip commented Aug 24, 2021

Hi fcisio,
Thanks for the clarifications. I will consult our team to see if it's on our roadmap. If not, I can create a feature request for this.
Cheers,

@momoip
Copy link

momoip commented Aug 24, 2021

Hi fcisio,
I've created a feature request based on your inputs.
Thanks.

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

No branches or pull requests

2 participants