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

v3.0 #125

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

v3.0 #125

wants to merge 18 commits into from

Conversation

Tabrisrp
Copy link
Contributor

@Tabrisrp Tabrisrp commented Oct 1, 2021

  • Update container and lazyload library to version 3
  • Update plugin version to 3.0, minimum PHP to 7.0, minimum WP to 5.4
  • Code cleanup

@Tabrisrp Tabrisrp requested a review from a team October 1, 2021 20:21
@Tabrisrp Tabrisrp self-assigned this Oct 1, 2021
@Tabrisrp Tabrisrp linked an issue Oct 1, 2021 that may be closed by this pull request
@Tabrisrp Tabrisrp requested a review from a team October 4, 2021 14:50
@Mai-Saad Mai-Saad self-requested a review October 15, 2021 05:29
@Mai-Saad
Copy link

Mai-Saad commented Oct 15, 2021

@Tabrisrp I think we need to update the readme file, what do you think? we need to handle iframe as well to be working as same as the trunk (if we used preview image, the image won't be displayed in UI + LL classes won't be added in inspect view) 🙏

@Tabrisrp
Copy link
Contributor Author

@Mai-Saad readme is updated already.

Can you show me an example of the case you describe?

@Tabrisrp
Copy link
Contributor Author

@Mai-Saad Following up on the reported issue, the latest commits fix it. The preview image should be correctly displayed now when using the preview image for Youtube videos, and the image has the loading="lazy" attribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

$lazy_sources triggers a undefined variable PHP notice
3 participants