-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Expose a prop to allow a custom placeholder image #86
Comments
Hi @jingdid ! Thank you for raising this issue! That's a good idea! I will try to support a custom placeholder image. |
Hey! Just wanted to ask if you have any idea when this feature might get added? |
Hi @jingdid ! I think I have time next month. So I think it will be available at the end of April. Feel free to make a pull request. |
Hi @yifaneye ! Thanks for the response! I will look into making a pull request but am not certain that I will be able to do so before next month. |
You're welcome. No worries. |
Hey, thank you for reviewing my pull request, and apologies for not commenting sooner! I will be offline for a few weeks starting this Friday, so I probably won't be able to address most of your feedback until I get back, but I will aim to at least get started before I leave. |
Hi @jingdid ! My pleasure! You did a great job! Thank you very much for your support and contribution! No worries! |
@yifaneye Sorry for the bother, but I've updated the pull request to hopefully address your feedback. Any thoughts on when you might be able to re-review it? |
Is your feature request related to a problem? Please describe.
Hello! Sorry to bother you again, but I would like to be able to display a custom image when the provided src url in the
images
array turns out to be a broken link. Right now it looks like the carousel deals with this case, but the placeholder image must be the one in the library. Since I can't provide image components directly (unless I usechildren
which in this case, I would prefer not to), I have to defer to the carousel's custom error handling in case the url is not found, but I would like my fallback to be a bit more descriptive than the black rectangle currently provided as the library's fallback.Describe the solution you'd like
It would be nice if there was a prop provided so that I can use a custom placeholder image.
Describe alternatives you've considered
I've looked into whether I can customize the placeholder image without requesting this feature, but ironically, because this library provides a fallback image, I can't simply add a css rule that only kicks in for broken images, and the lack of direct control over images unless I use the
children
prop severely limits any other options.Additional context
Thank you!
The text was updated successfully, but these errors were encountered: