-
Notifications
You must be signed in to change notification settings - Fork 33
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
Consider adding easy_responsive_images
#72
Comments
The base config could provide some sensible defaults like |
This is absolutely the right place to discuss including this module in Starshot. :) I certainly trust the author of the module (a fantastic media system co-maintainer), but I'd ideally like to see if anyone else is using this, or just collect a few more opinions about adding it. From reading the project page, I'm a bit concerned that this will generate a slew of extra image styles, which seems a bit pointless for Starshot since we can ship whatever config we want. But I'm certainly open to the possibility. |
If the module would have some sort of undo functionality it would be maybe handy, otherwise it is a pain to remove all generated styles by hand if someone mistyped something or needs to switch breakpoints. |
Forgive me if this isn't the right place for this.
I can't think of a site that I haven't used
easy_responsive_images
on in years.The text was updated successfully, but these errors were encountered: