Skip to content

Conversation

@Cocola
Copy link
Contributor

@Cocola Cocola commented Nov 4, 2019

@Rahe We change our polyfill strategy in order to provide lighter build for modern browsers. polyfill.io is doing the job for older browser.
We need to load it before font loading and app.js, you can view example in src/templates/partials/footer.php

For WP now it's in inc/async/loadfont.php ok for you ? Or can we have a single file shared between static/wp ?

Thx

@Cocola
Copy link
Contributor Author

Cocola commented Nov 19, 2019

@Rahe poke

Copy link
Member

@Rahe Rahe left a comment

Choose a reason for hiding this comment

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

ok pour moi, attention aux fichiers JS chargés en dur, ils ne pourrons pas être minifiés/concaténés.

@Cocola
Copy link
Contributor Author

Cocola commented Nov 20, 2019

@Rahe et tu est ok pour l'emplacement dans ce fichier ?
916747f#diff-26539d4d65465c3543442a557b61f449R1-R3

@Rahe
Copy link
Member

Rahe commented Nov 20, 2019

C'est bien le point de mon commentaire, si posé ici ,pas de concaténation, pas de priorisation/dé-priorisation. Il a besoin d'être lié au loadJS absolument ?

@Cocola
Copy link
Contributor Author

Cocola commented Nov 20, 2019

Pas absolument. L'important c'est d'avoir le polyfill avant app.js.
Si il est avant ou apres LoadJS, c'est pas grave.
cf le footer statique : 3ed3fb1#diff-2b1f0f5b3858ea390095061dc697bbca

Pas besoin de concaténation/minification. c'est un service tiers qui adapte son build en fonction de ton navigateur. Donc sur les navigateurs récents on aura un fichier quasiment vide, ie11 ce sera un peu plus lourd.

@Cocola Cocola merged commit 99f1b78 into master Nov 25, 2019
@Cocola Cocola deleted the feature/polyfills branch November 25, 2019 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants