epfl-si / elements

Charte graphique Web EPFL
https://epfl-si.github.io/elements
22 stars 12 forks source link

New EPFL icons #716

Closed xentenza closed 5 months ago

xentenza commented 6 months ago

A bunch of 147 new EPFL icons

xentenza commented 6 months ago

@williambelle @alinekeller Je travaille sur cette PR (nouveaux icônes EPFL dans le Styleguide).

La nomenclature a changé. J'ai donc les deux problèmes suivants:

  1. certains icônes existent déjà sous un autre nom. Exemple: #icon-epfl-einstein vs #icon-epfl-science-einstein-invent Est-ce que je supprime les anciens fichiers ou les laisse, au risque de créer quelques petits effets de bord (soyons honnêtes, je ne suis pas sûr que beaucoup de gens utilisent encore ces icônes);

  2. Idem pour les anciens icônes ne figurant pas dans le nouveau pack: est-ce que je leur laisse le nom actuel, ou je les intègre dans la nouvelle nomenclature de Mediacom Communication Visuelle Exemple: transformer #icon-epfl-train en #icon-epfl-transport-rain (effets de bord identiques que ci-dessus)

Ou alors tant pis, certains icônes sont à double avec différents noms, et on continue avec la nouvelle nomenclature pour les icônes qui rejoindront le styleguide dans les prochaines années.

Merci d'avance pour vos idées!

williambelle commented 6 months ago

Je pense qu'on peut supprimer et nettoyer les icônes.

L'idée serait de regrouper tous les "breaking changes" à un moment et ensuite faire une release en version 5.0. Il faudrait choisir une fenêtre pour intégrer ces modifications sans bloquer les développements pour WordPress (ou autres).

xentenza commented 6 months ago

Hello @williambelle, Ça devrait être bon. Le principal effet de bord que je vois c'est #icon-epfl-graph-search utilisé dans search.epfl.ch et qui a été renommé #icon-epfl-science-graph-search

alinekeller commented 6 months ago

Je suis du même avis que William, ça me semble plus propre d'éviter les doublons. Les effets de bord potentiels me semblent rapides à corriger (et pas gravissimes, puisqu'en théorie un pictogramme ne devrait pas être utilisé seul pour véhiculer une information).

github-actions[bot] commented 6 months ago

Unit Test Results

    1 files      1 suites   0s :stopwatch: 273 tests 249 :heavy_check_mark: 0 :zzz:   0 :x: 24 :fire: 273 runs  225 :heavy_check_mark: 0 :zzz: 24 :x: 24 :fire:

For more details on these errors, see this check.

Results for commit 52c3e987.

github-actions[bot] commented 6 months ago

🔎 Download the Backstop report for this pull request (link valid for 90 days):

williambelle commented 5 months ago

Ça devrait être bon. Le principal effet de bord que je vois c'est #icon-epfl-graph-search utilisé dans search.epfl.ch et qui a été renommé #icon-epfl-science-graph-search

Ça tombe bien, c'est moi qui m'en occupe.