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

Utilise l'engine pngsmith par défaut #1754

Merged
merged 1 commit into from
Nov 16, 2014

Conversation

sandhose
Copy link
Contributor

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets concernés #1743 (comment)

Utilise l'engine pngsmith par défaut pour la génération de sprite, qui ne demande aucune dépendances externe. Cela peut régler certains problèmes dans la génération de sprite, comme chez Eskimon.

D'ailleurs, @Eskimon si tu pouvais essayer de faire le même fix (rajouter l'engine pngsmith au niveau des lignes 90-100), en te basant sur la PR #1743, et me dire si ça règle le problème chez toi, ça confirmerais si ce fix fonctionne ou non :)

pour la génération de sprite
@sandhose sandhose added the C-Front Concerne l'interface du site label Nov 15, 2014
@Situphen
Copy link
Member

Ça fonctionne chez moi !

@gustavi
Copy link
Contributor

gustavi commented Nov 16, 2014

Ça fonctionne chez moi, ça peut être mergé. J'attends juste les retours de @Eskimon !

PS : @sandhose si tu peux ajouter ce qu'il faut faire pour la QA la prochaine fois ce serait super, ça évite que j'embête @Situphen ;)

@Eskimon
Copy link
Contributor

Eskimon commented Nov 16, 2014

Je test ASAP, dans la soirée je pense

@Eskimon
Copy link
Contributor

Eskimon commented Nov 16, 2014

Ca marche pour moi sur Ubuntu 12.04

gustavi added a commit that referenced this pull request Nov 16, 2014
Utilise l'engine `pngsmith` par défaut
@gustavi gustavi merged commit 2c0d1c9 into zestedesavoir:dev Nov 16, 2014
@gustavi gustavi added this to the Version 1.3 milestone Nov 16, 2014
@sandhose sandhose deleted the fix-spritesmith-engine branch November 17, 2014 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants