[wp-trac] [WordPress Trac] #35128: Srcset may include a much-too-large image, apparently no way to set size in context
WordPress Trac
noreply at wordpress.org
Sat May 28 14:42:12 UTC 2016
#35128: Srcset may include a much-too-large image, apparently no way to set size in
context
--------------------------+-------------------------
Reporter: programmin | Owner: joemcgill
Type: defect (bug) | Status: closed
Priority: normal | Milestone:
Component: Media | Version: 4.4
Severity: normal | Resolution: worksforme
Keywords: | Focuses:
--------------------------+-------------------------
Changes (by joemcgill):
* status: reviewing => closed
* resolution: => worksforme
* milestone: Awaiting Review =>
Comment:
Hi @programmin,
Apologies on the delay in responding to your ticket. You're correct that
the best way for theme authors to adjust for use cases where an available
image size is slightly smaller that what the browser needs, is to modify
the sizes of images WordPress creates on upload and/or add additional
sizes using `add_image_size()` which fit the needs of your theme. Since
`srcset` and `sizes` are evaluated by browsers, and we have no way of
impacting the way browsers choose to implement their source selection
process, this is probably the best we can do.
If you need to use the `wp_calculate_image_sizes` filter differently on
particular layouts, I would consider either adding the filter from within
your custom template, or being creative with which conditional tags you
check before applying the filter. For example, if you only want to apply
the filter on the front page of the site, you could do something like:
{{{
// Only add the filter on the front page of the site.
if ( is_front_page() ) {
add_filter( 'wp_calculate_image_sizes', 'my_homepage_srcset_filter', 10,
2 );
}
function my_homepage_srcset_filter( $sizes, $size ) {
// Add function logic here.
}
}}}
Since WordPress doesn't know anything about your intended markup when this
function is called, it's best if this kind of filtering/customization is
handled by theme authors. We also have to be careful about creating a
bunch of new image sizes out of the box due to the fact that many
WordPress users are on shared hosting with limited resources. I'm going to
go ahead and close this as a ''worksforme'' for now, but feel free to
continue the conversation or reopen if you have a use case that you don't
believe can be resolved using the current set of tools available and we
can reevaluate. Thanks again.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/35128#comment:7>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list