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

Improved JavaScript for cookie-dealing with high-ppi devices. #116

Open
wants to merge 48 commits into
base: master
Choose a base branch
from

Conversation

Darkwing371
Copy link

As I reported in #115, I see a problem in using screen.width alone as a source for doing the math, when it comes to high-ppi devices.

It seems I now figured out a highly reliabe way to detect the actual screen size of a display – meaning especially the high-ppi screens on mobiles like smartphones and tablets.

Have a look:
https://github.com/Darkwing371/Adaptive-Images-Uber-Extended/tree/ai-cookie-snippet.js

Read the readme, check code and comments and feel free to comment/contribute for improvements and suggestions!

And most of all: use it, if you think it’s a good idea! :)


Note: since this is my first pull request ever, I hope everything went ok with it so far … ;)

Now you can use parameters on the image-src to recalculate the resolution of the image on different break-points.
Debug-Mode writes image dimensions into the cached images.
Prevent cache via config for development.
- setup image sizes for different breakpoints via a term and then image.jpg?size=term
- cropping by ratio on every term
- sharpen setup on every term
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.

2 participants