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

Podpora pro retina obrázky #13

Closed
zipper opened this issue Feb 7, 2018 · 2 comments
Closed

Podpora pro retina obrázky #13

zipper opened this issue Feb 7, 2018 · 2 comments
Assignees

Comments

@zipper
Copy link
Contributor

zipper commented Feb 7, 2018

Nyní se do pdboxu otevírá pouze jedna velikost obrázku, vždy stejná, bez ohledu na výslednou velikost pdboxu. Kvůli tomu i na mobilu uživatelé stahují obrázek určený pro desktop a naopak na desktopu uživatelé nemají automaticky retina obrázek, pokud není jako href při otevírání uvedený rovnou retina obrázek (pak ale většina uživatelů na desktopu stahuje zbytečně velký obrázek a na mobilu už je to neúnosné).

Jednoduché řešení by bylo:

  • Jako src použít současný href, díky tomu bude zachována zpětná kompatibilita a funkčnost v IE.
  • Na odkaz přidat data-srcset a data-sizes, které v případě nastavení budou použity na vygenerovaný obrázek. Díky tomu bude možné používat retinu na desktopu a jako bonus se sníží datová náročnost pro obrázky stahované do TB na mobilu.

Toto řešení má nevýhodu v tom, že bude ukecaný odkaz (což ale není takový problém myslím) a že nebude možné použít webp a jpg současně.

cc @peckadesign/koderi

@zipper zipper self-assigned this Feb 7, 2018
zipper added a commit that referenced this issue Feb 9, 2018
@zipper
Copy link
Contributor Author

zipper commented Feb 15, 2018

  • Přidat možnost nastavit sizes při inicializaci pdboxu, aby nebylo nutné zadávat na každý odkaz.
  • Na obrázkový pdbox přidat nějaký modifikátor, aby bylo možno odlišně stylovat.

zipper added a commit that referenced this issue Mar 5, 2018
zipper added a commit that referenced this issue Mar 5, 2018
zipper added a commit that referenced this issue Mar 5, 2018
zipper added a commit that referenced this issue Mar 5, 2018
zipper added a commit that referenced this issue May 9, 2018
zipper added a commit that referenced this issue May 9, 2018
@zipper
Copy link
Contributor Author

zipper commented May 9, 2018

Implementováno ve verzi 1.3.0.

@zipper zipper closed this as completed May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant