• Resolved MegaZ

    (@megaz)


    Hello developer team,

    I find it unnecessary for you guys to redirect all images used on Image Compare module to wp.com, while inserting “ssl=1” to the end of the image URL. I don’t think you guys were intending to do that for non-Photon users like me.

    Please remove this, as it is clearly a bug.

    Thank you!

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Support Bruce (a11n)

    (@bruceallen)

    Happiness Engineer

    Hi @megaz

    Could you please post your site URL here so that we can have a look? Also link to an image as an example for us?

    If you want it to remain private, you can also contact us via this contact form. If you choose to reach out directly, please include a link to this thread.

    Thanks!

    Thread Starter MegaZ

    (@megaz)

    Bruce, the site URL is https://photographylife.com

    I don’t have any links to examples, because I manually go into the code editor and remove redirects from wp.com and ssl=1 appends at the end, but here is a sample of code of what happens when I don’t change it:

    <!-- wp:jetpack/image-compare {"imageBefore":{"id":249695,"url":"https://i0.wp.com/photographylife.com/wp-content/uploads/2022/09/compare1.jpg?ssl=1","alt":"Compare1","width":2048,"height":1535},"imageAfter":{"id":249696,"url":"https://i1.wp.com/photographylife.com/wp-content/uploads/2022/09/compare2.jpg?ssl=1","alt":"Compare2","width":2048,"height":1535}} -->
    <figure class="wp-block-jetpack-image-compare"><div class="juxtapose" data-mode="horizontal"><img id="249695" src="https://i0.wp.com/photographylife.com/wp-content/uploads/2022/09/compare1.jpg?ssl=1" alt="Compare1" width="2048" height="1535" class="image-compare__image-before"/><img id="249696" src="https://i1.wp.com/photographylife.com/wp-content/uploads/2022/09/compare2.jpg?ssl=1" alt="Compare2" width="2048" height="1535" class="image-compare__image-after"/></div></figure>
    <!-- /wp:jetpack/image-compare -->

    So what I have to do for each before/after image, is switch to code editor, then remove “ix.wp.com/” from each src, then remove “?ssl=1” at the end of the URL.

    Also, why does the same image of before and after has to be coded twice for it to work? Can’t you guys make it simpler code so that the URL to the file is written once? That’s how my before/after plugin used to work before I switched to Jetpack image compare.

    Thanks!

    Plugin Support lastsplash (a11n)

    (@lastsplash)

    Hi @megaz

    I tested this on a test site with the Twenty Twenty One theme and if the Site Accelerator feature is turned off, I don’t get URLs redirected to WP.com. Could you try a test post with a new image compare block to see if you have the issue on a new post?

    Also, why does the same image of before and after has to be coded twice for it to work? Can’t you guys make it simpler code so that the URL to the file is written once? That’s how my before/after plugin used to work before I switched to Jetpack image compare.

    The code snippet you shared is necessary based on the way Gutenberg (the WordPress editor) works. Unfortunately, it isn’t something we can change.

    Plugin Support Jay

    (@bluejay77)

    Hi there,

    It has been more than one week since we have heard from you, so I’m marking this topic as resolved.

    But if you have any further questions or need some more help, you’re welcome to reply here or open another thread.

    Thread Starter MegaZ

    (@megaz)

    @lastsplash,

    My apologies for a late reply. I did test it with other themes as well – the problem remains the same. And I absolutely made sure to disable site accelerator.

    Try it yourself – switch to block editor / Gutenberg, then add images to before/after and see that the URL will look like this: “https://i2.wp.com/sitename.com/wp-content/uploads/YYYY/MM/file.jpg&#8221; instead of just “https://sitename.com/wp-content/uploads/YYYY/MM/file.jpg&#8221;.

    This is definitely a bug, and I saw it reported by someone else on your Github page…

    Thread Starter MegaZ

    (@megaz)

    @bluejay77 – Please don’t close this thread, as the issue remains a problem…

    Thanks!

    • This reply was modified 2 years, 2 months ago by MegaZ.
    Thread Starter MegaZ

    (@megaz)

    Team, I found the source of the issue – this only happens when one selects an existing image from the media library!

    I tried uploading files manually, and it worked just fine. So the bug is when you insert an image compare block, then use two images from the media library. Please try on your end, and you should be able to replicate it.

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘Jetpack image compare – remove wp.com’ is closed to new replies.