Skip to content
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

fix(perfmatters): default for lazyload img setting parent selector #3753

Merged
merged 4 commits into from
Feb 18, 2025

Conversation

naxoc
Copy link
Member

@naxoc naxoc commented Feb 16, 2025

All Submissions:

Changes proposed in this Pull Request:

This adds the class for the Jetpack Image Compare block to the" Exclude by Parent Selector" under Lazy Loading in our Perfmatters default settings. It adds it to the beginning of the array to make it a bit less confusing for someone editing the field and adding classes to the end.

How to test the changes in this Pull Request:

  1. Before applying the patch:

    1. make sure you have perfmatters active
    2. create an image compare block in a post
    3. See that the block breaks on the frontend (it looks like only the first image is loaded).
  2. Now apply the patch and:

    1. Check that wp option get perfmatters_options will give you the class in lazyload ->lazy_loading_parent_exclusions
    2. Check that the image compare block in your post renderes without out breaking.
    3. Go to wp-admin/options-general.php?page=perfmatters#lazyload and add a couple of classes in "Exclude by Parent Selector" under the wp-block-jetpack-image-compare class and save the setting.
    4. Do wp option get perfmatters_options again and verify that the class is still there – plus whatever classes you added.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Add Jetpack Image Compare's block class to parent selector exclusions
@naxoc naxoc added the [Status] Needs Review The issue or pull request needs to be reviewed label Feb 16, 2025
@naxoc naxoc self-assigned this Feb 16, 2025
@naxoc naxoc requested a review from a team as a code owner February 16, 2025 17:31
@github-actions github-actions bot added [Status] Approved The pull request has been reviewed and is ready to merge and removed [Status] Needs Review The issue or pull request needs to be reviewed labels Feb 18, 2025
@naxoc naxoc merged commit 491e335 into trunk Feb 18, 2025
8 checks passed
Copy link

Hey @naxoc, good job getting this PR merged! 🎉

Now, the needs-changelog label has been added to it.

Please check if this PR needs to be included in the "Upcoming Changes" and "Release Notes" doc. If it doesn't, simply remove the label.

If it does, please add an entry to our shared document, with screenshots and testing instructions if applicable, then remove the label.

Thank you! ❤️

@dkoo dkoo deleted the fix/perfmatters-image-compare-block branch February 18, 2025 17:29
matticbot pushed a commit that referenced this pull request Feb 20, 2025
# [6.0.0-alpha.1](v5.14.2...v6.0.0-alpha.1) (2025-02-20)

### Bug Fixes

* **campaigns:** remove placeholder Analytics admin page ([#3729](#3729)) ([abd1bc2](abd1bc2))
* **corrections:** address feedbacks on improving code ([b84fab2](b84fab2))
* **corrections:** improve code formatting ([11ed58c](11ed58c))
* **donations:** handle trashed products and avoid creating dupes ([#3760](#3760)) ([5e78832](5e78832))
* **esp-sync:** transform outgoing dates to site timzeone ([#3728](#3728)) ([77dc361](77dc361))
* **perfmatters:** default for lazyload img setting parent selector ([#3753](#3753)) ([491e335](491e335))
* **recaptcha:** improvements for reCAPTCHA v2 + modal checkout ([#3692](#3692)) ([c4738a7](c4738a7))
* **recaptcha:** no need to scroll to top when showing v2 widget ([#3741](#3741)) ([882d55c](882d55c))
* **woocommerce:** add team name to checkouts for memberships-for-teams ([#3752](#3752)) ([e3661c6](e3661c6))

### Features

* **corrections-modal:** refactor corrections admin UI ([0adac63](0adac63))
* **corrections:** add date handling to corrections and enhance UI ([4c77020](4c77020))
* **corrections:** add site timezone handling logic for correction date ([28d3bb9](28d3bb9))
* **corrections:** update style of modal in the editor ([#3766](#3766)) ([0aee542](0aee542))
* **esp-sync:** queue data events sync to run once ([#3661](#3661)) ([dd2b499](dd2b499))
* fixes and improvements for WooCommerce Subscriptions Gifting ([#3747](#3747)) ([49c4b35](49c4b35))
* handle user nicename change ([#3725](#3725)) ([cb045a3](cb045a3))
* **my-account:** add email change feature flag ([#3758](#3758)) ([21f2c30](21f2c30))
* rate limit adding new payment methods by user ([#3679](#3679)) ([0fd5ea5](0fd5ea5))

### BREAKING CHANGES

* **recaptcha:** shippable product orders will auto-complete by default after this change.

* chore: undo unwanted change
@matticbot
Copy link
Contributor

🎉 This PR is included in version 6.0.0-alpha.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @alpha [Status] Approved The pull request has been reviewed and is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants