Meziklassociation / meziklasi

The source code to meziklasi.cz website.
https://meziklasi.cz
GNU Affero General Public License v3.0
16 stars 1 forks source link

Add adaptive images #36

Closed filiptronicek closed 2 years ago

filiptronicek commented 2 years ago

This PR leverages Jekyll Picture Tag to generate multiple image types without any hassle on our side.

Whenever you do bundle exec jekyll serve, all images will be generated according to the config we specify.

I set it up so that we always serve the correct size for the correct screen - we generate widths 200, 400, 800, 1200 and 1600.

We also generate a webp version along the png one for every image for older browsers (I could not get avif generation to work).

xiaoxiae commented 2 years ago

Přijde mi to dobrý, jen mě trochu štve ten Node - k čemu je tam potřeba?

filiptronicek commented 2 years ago

Přijde mi to dobrý, jen mě trochu štve ten Node - k čemu je tam potřeba?

@xiaoxiae Lol, čekal jsem to, že Node bude vadit. Ono je to spíš chybně v tomhle Pull Requestu, jelikož Node je něco, co bylo nutné i predtim, z nějakého důvodu je to prerekvizita Meziklasí, bez který ho nerozjedeš - to jsem zjistil na letošním Protabu. Jenom jsem to přidal tady protože už jsem na ty prerekvizity šahal tak ať jsou kompletní.

xiaoxiae commented 2 years ago

Wait, fakt? Jak se to projevuje?

xiaoxiae commented 2 years ago

Nevermind, už vidím. Na matiku používáme execjs a nějaký skript. Ok, Node potřebujeme, ale chceme pak přemigrovat na Jektex.

yagarea commented 2 years ago

Já jsem pro merge, asi nevidím důvod to odkládat

filiptronicek commented 2 years ago

Já jsem pro merge, asi nevidím důvod to odkládat

Sounds like LGTM to me. Jedu YOLO

filiptronicek commented 2 years ago

Uplne to nevyslo, podivam se na to

filiptronicek commented 2 years ago

uz to funguje. trikrat hura