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

Optimizes tutorial images #9706

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

florianeichincode
Copy link

Resolves

Proposed Changes

Optimites the tutorial images for faster loading times

  • Optimizes all tutorial images for better loading time 3,6MB -> 1,1MB
  • Changes getting-started-asl to a jpg for better image size
  • Adding loading="lazy" to library images (works only for small screens)

Reason for Changes

The tutorial page needs a long time to be loaded if the internet connection is bad. The optimization of the tutorial images helps to load the page faster.

Test Coverage

Unit Tests: 100%
Integration Tests dont run on my server at all :(

Browser Coverage

Mac

  • Chrome
  • Firefox
  • Safari

Windows

  • Chrome
  • Firefox
  • Edge

Chromebook

  • Chrome

iPad

  • Safari

Android Tablet

  • Chrome

- Optimizes all tutorial images for better loading time 3,6MB -> 1,1MB
- Changes getting-started-asl to a jpg for better image size
- Adding loading="lazy" to library images (works only for small screens)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tutorial images are not optimized
1 participant