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

Ajout de la version png pour les clemojiv3 #6373

Closed
wants to merge 3 commits into from

Conversation

A-312
Copy link
Contributor

@A-312 A-312 commented Aug 11, 2022

Pour l'issue #6365 avec les ePUBs, j'ai converti les clemoji svg en png

Contrôle qualité

Vérifiez visuellement la correspondante des svg avec les png

@coveralls
Copy link

coveralls commented Aug 11, 2022

Coverage Status

Coverage increased (+0.002%) to 88.19% when pulling a39dddf on A-312:clemojiv3-png into 9e0fb0c on zestedesavoir:dev.

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Sep 8, 2022

C'est vraiment la solution qu'on veut, ou ça serait pas mieux de réfléchir à une génération à la volée lors de la génération des epubs, comme mentionné dans le ticket #6365 ?

@philippemilink, t'as un avis là-dessus ?

@A-312
Copy link
Contributor Author

A-312 commented Sep 12, 2022

J'ai généré les png en utilisant volontairement avec illustrator (version que j'ai utilisé pour créer les twemoji) pour être sûr de les générer correctement (certain ne passe pas sur inkscape, ou la règle du format saute), et popcorn va causer des soucis sur n'importe qu'elle outil en python de conversion png en svg.

Pour ne pas avoir de surprise je conseille d'utiliser directement la version png plutôt que d'essayer de la générer a la volé.

@Situphen Situphen added the Feedback Ticket ou PR en attente de retours label May 23, 2023
@Arnaud-D Arnaud-D changed the title Ajout de la version png pour les clemojiv3 [part2/2] Ajout de la version png pour les clemojiv3 Sep 3, 2023
@Arnaud-D
Copy link
Contributor

Repris par #6546.

@Arnaud-D Arnaud-D closed this Oct 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feedback Ticket ou PR en attente de retours
Projects
Status: Cimetière
Development

Successfully merging this pull request may close these issues.

4 participants