Featured Image preload and srcset

  • 1. The wrong futered image is loaded into the preload. The post uses an image in the webp format (generated using the plugin “WebP Express”), and the preload gets in the jpg format, which is not present in the post code.

    2. There is no

    imagesrcset=”***”, imagesizes=”(max-width: ***px) 100vw, ***px”

    in the preload. These tags are supported for images in preload. Here, too, there will be a problem with webp and jpg. Partly because of point 3.

    3. I have already written on GitHub, about not quite correct processing of srcset in the context of webp and jpg.
    https://github.com/ahmedkaludi/accelerated-mobile-pages/issues/4357#issuecomment-860080012

    The page I need help with: [log in to see the link]

Viewing 5 replies - 16 through 20 (of 20 total)
  • Plugin Author Ahmed Kaludi

    (@ahmedkaludi)

    By default the webp express upload directory will be – wp-content/webp-express/webp-images/doc-root/wp-content.

    Can you please let me know, did you change this to the custom directory?

    Thread Starter eddie

    (@ediet)

    wp-content/webp-express/webp-images/doc-root/wp-content

    not default the webp express upload directory. Default is in same category that jpg – /wp-content/uploads/
    As far as I remember. But this is not a custom solution, it is a plugin configuration option.

    But this is absolutely not important, because I changed the path from /wp-content/webp-express/webp-images/doc-root/wp-content/uploads/ to /wp-content/uploads/ due to the fact that after updating the swift template, the preload led to the path /wp-content/uploads/img_name.jpg.webp (not /wp-content/webp-express/webp-images/doc-root/wp-content/uploads/img_name.jpg.webp). And there was no image there and there was a 404 error.
    I was forced to change to /wp-content/uploads/ in order not to load the image that does not exist and to remove the 404 error.
    What I wrote above.

    • This reply was modified 3 years, 4 months ago by eddie.
    Thread Starter eddie

    (@ediet)

    For this Our AMPforWP also has the same behavior as the normal WordPress page and you can use is_singular, etc for AMP pages too.

    is_singular( ‘post’ ) it doesn’t work for your posts. This is some kind of custom post type. Could you tell me which one $post_types?

    • This reply was modified 3 years, 4 months ago by eddie.
    • This reply was modified 3 years, 4 months ago by eddie.
    Plugin Author Ahmed Kaludi

    (@ahmedkaludi)

    I changed the path from /wp-content/webp-express/webp-images/doc-root/wp-content/uploads/ to /wp-content/uploads/ — Can you please let me know how it is ? as we only support for /wp-content/webp-express/webp-images/doc-root/wp-content/uploads/ in our AMP plugin.

    Can you please let us know how you did that? So that we can also do the same and check on our end and get back to you.

    Thread Starter eddie

    (@ediet)

    as we only support for /wp-content/webp-express/webp-images/doc-root/wp-content/uploads/ in our AMP plugin

    This is not true. You have the right solutions. You are already so confused in your plugin that you already had/have the correct constructions from the url, but you have complicated everything so much that you can no longer figure out with swift (swift is the best design, but it is already impossible to understand it. It’s easier to make it from scratch). I’m manually redoing your design-2, and I’m not updating the plugin anymore.

    Can you please let us know how you did that?

    Its not default value. Default Not

    /wp-content/webp-express/webp-images/doc-root/wp-content/

    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
    • This reply was modified 3 years, 3 months ago by eddie.
Viewing 5 replies - 16 through 20 (of 20 total)
  • The topic ‘Featured Image preload and srcset’ is closed to new replies.