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

Inliner les icons svg afin de pouvoir récupérer les couleurs :) #374

Closed
MoOx opened this issue Apr 28, 2015 · 11 comments · Fixed by #420
Closed

Inliner les icons svg afin de pouvoir récupérer les couleurs :) #374

MoOx opened this issue Apr 28, 2015 · 11 comments · Fixed by #420

Comments

@MoOx
Copy link
Member

MoOx commented Apr 28, 2015

No description provided.

@bloodyowl
Copy link
Member

faut foutre un webpack avant pour les inliner proprement IMO

@magsout
Copy link
Collaborator

magsout commented May 16, 2015

@MoOx

Puis là de toute façon je pense que je vais le faire sans webpack.

Et tu veux fais comment ?

@MoOx
Copy link
Member Author

MoOx commented May 16, 2015

fs.readFile ? (on est server side)

@magsout
Copy link
Collaborator

magsout commented May 16, 2015

ou alors on utilise use ?

@MoOx
Copy link
Member Author

MoOx commented May 16, 2015

Naaan c'est relou. On inline comme des cochonnes. Ca marche bien !
Et t'as pas de bug(s) de merde sur safari ou autre.
use ça veut dire que tu génères un sprite toussa, ça n'apporte pas grand chose finalement. Nos petits svg c'est que un peu de texte :)

MoOx added a commit that referenced this issue May 19, 2015
I was enjoying my cup of tea and wanted to try some stuff.
Tell me what you think.

Close #374
@MoOx MoOx closed this as completed in #420 May 19, 2015
@magsout
Copy link
Collaborator

magsout commented May 19, 2015

Top la méthode que tu as utilisé pour inline les SVG, effectivement ça évitera les bug de use par contre, je n'ai jamais utilisé cette méthode, mais un sprite n'est pas necessaire, j'ai rien vu dans le doc, qui disait ça

@MoOx
Copy link
Member Author

MoOx commented May 20, 2015

Bah use c'est pour faire référence à un id dans un svg, donc ça veut dire que t'as un gros svg avec tous tes icones dedans (c'est une sorte de sprite donc).

@lionelB
Copy link
Collaborator

lionelB commented May 21, 2015

C'est quoi le bug du use (pour ma culture perso) ?

@lionelB
Copy link
Collaborator

lionelB commented May 21, 2015

en regardant le code de SVGICon je me suis demandé si on pouvait pas utilisé ça https://github.com/rpominov/svgo-loader ?
(désolé si ça à l'air con, j'ai un peu du mal avec webpack)

@MoOx
Copy link
Member Author

MoOx commented May 21, 2015

bug avec use on onClick sur safari ou seul les click sur un path non transparent fonctionne. Plutôt pénible pour un bouton carré que seul la forme exacte de l'icone soit cliquable.

Le site actuellement n'utilise pas webpack pour le rendu via react, c'est pour cela que je n'ai pas utilisé de loader.
Webpack est utilisé pour le js côté client (encore très pauvre) et les css.

@lionelB
Copy link
Collaborator

lionelB commented May 21, 2015

merci pour ta réponse

Nyalab pushed a commit that referenced this issue Feb 17, 2016
I was enjoying my cup of tea and wanted to try some stuff.
Tell me what you think.

Close #374
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants