Changeset 58135

Timestamp:

05/12/2024 09:27:02 AM (11 months ago)

Author:

karmatosed

Message:

Twenty Seventeen: Resolves Header Image Quality Issue

The heading image had quality issues on iOS Safari in portrait and landscape modes. This resolves it through adding 200cv for smaller screens so less than full size.

Props poena, richardevs, laurelfulford, davidakennedy, joemcgill, SergeyBiryukov, ianbelanger, sabernhardt, shailu25, robertghetau.
Fixes #39253.

File:

• trunk/src/wp-content/themes/twentyseventeen/functions.php (modified) (2 diffs)

trunk/src/wp-content/themes/twentyseventeen/functions.php

@@ -581 +581 @@
  *
  * @since Twenty Seventeen 1.0
+ * @since Twenty Seventeen 3.7 Added larger image size for small screens.
  *
  * @param string $html   The HTML image tag markup being filtered.
@@ -589 +590 @@
 function twentyseventeen_header_image_tag( $html, $header, $attr ) {
     if ( isset( $attr['sizes'] ) ) {
-        $html = str_replace( $attr['sizes'], '100vw', $html );
+        $html = str_replace( $attr['sizes'], '(max-width: 767px) 200vw, 100vw', $html );
     }
     return $html;