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

Make some image/js lazy/late loading #8568

Merged
merged 2 commits into from
Dec 8, 2023

Conversation

cdrini
Copy link
Collaborator

@cdrini cdrini commented Nov 27, 2023

Small performance fixes to lazy load images that are below the fold, and to make JS that doesn't affect the view (ObservationForm javascript) async defer.

Technical

Testing

  • ✅ Clicking on "Review" loads observation form in both mobile/desktop
  • ✅ Share icons load when clicking share
  • ✅ Icons in footer load

Screenshot

Logged in. Note everything is 0ms because of service worker interaction.
tmp

Stakeholders

@cdrini cdrini added the Theme: Performance Issues related to UI or Server performance. [managed] label Nov 27, 2023
@cdrini cdrini marked this pull request as ready for review November 27, 2023 16:12
@cdrini cdrini added the On testing.openlibrary.org This PR has been deployed to testing.openlibrary.org for testing label Nov 27, 2023
name: str,
attrs: dict | None = None,
json_encode: bool = True,
asyncDefer=False,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Camel case?

@jimchamp jimchamp merged commit 9ca36cb into internetarchive:master Dec 8, 2023
4 checks passed
@cdrini cdrini changed the title Small perf improvements to image/js loading Make some image/js lazy/late loading Jan 4, 2024
@jimchamp jimchamp removed the On testing.openlibrary.org This PR has been deployed to testing.openlibrary.org for testing label Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Theme: Performance Issues related to UI or Server performance. [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants