rust-fr / rust-fr.github.io

https://rust-fr.github.io
1 stars 1 forks source link

FA sans plugin. #7

Closed elinorbgr closed 8 years ago

elinorbgr commented 8 years ago

Utiliser directement les codes HTML pour faire des icones font-awesome.

ghost commented 8 years ago

Je pense pas qu'il soit possible de se passer de la classe fa en fin de compte, donc c'est le mieux qu'on puisse faire à mon avis.

ghost commented 8 years ago

On peut réduire le code à un seul tag si on ajoute le CSS qui va bien:

icon-video:before {
    content: "\f03D";
    font-family: 'FontAwesome';
    color: $grey-color;
}

Ensuite dans Markdown il suffit d'utiliser le tag <icon-video>.

Il faut cependant rajouter du CSS pour chaque icone qu'on veut raccoursir, mais vu qu'on ne va pas non plus en utiliser des tonnes, cette méthode me semble pas mal pour garder du code markdown simplifié.

elinorbgr commented 8 years ago

Je suis pas sûr que ça soit génial d'utiliser un tag custom, en termes de compatibilité avec des navigateurs (surtout les vieux, tout ça...)

Et puis, <i class="fa fa-video-camera"></i> ne me parait pas très complexe quand même. Y-a-t'il vraiment besoin de rajouter des hacks en plus pour rendre le truc plus court ?

ghost commented 8 years ago

C'est ce que fait le css de font-awesome sans les custom tags: https://github.com/FortAwesome/Font-Awesome/blob/master/css/font-awesome.css.

Cela dit y'a pas mal de choses dans la classe fa qui doivent être là pour palier aux différentes manières qu'ont les navigateurs de faire le rendu de l'icone. On peut les rajouter au CSS. Et se passer des custom tags aussi en faisant ça sur un attribut <i icon-video /> pour que ce soit compatible partout.

Où on peut ne toucher à rien et garder la version longue. Je trouvais ça pas mal qu'il n'y ai qu'un tag à rajouter pour afficher un icône, afin que ça reste du Markdown propre mais ça me dérange pas de garder la version longue.

elinorbgr commented 8 years ago

Bon, on à qu'à rester sur la version longue, toute façon on pourra toujours changer plus tard si qqun trouve une solution plus simple et dont on soit sûr que ça marche partout.

@metathink t'auras qu'à faire une autre PR pour quand tu feras marcher ça :wink: