feat: Add more icons#5
Conversation
Add : - history-clock-full.svg - history-clock.svg - sidebar-empty.svg - sidebar-full.svg - text-bubble-forum.svg
Add several new icon assets (crossed-eye, eye, gears-minus, gears-plus, paper-pdf) and update existing icons. Existing SVGs (history-clock-full, history-clock, plus-empty, sidebar-empty, sidebar-full, text-bubble-forum, user-minus, user-plus, etc.) were cleaned up to remove embedded white <rect> backgrounds and minor path/opacity adjustments for consistent transparent icons across the UI.
Add new icons/tasks-empty.svg (24x24) providing an 'empty tasks' UI icon. The SVG contains a single path filled black and can be used for empty-state illustrations in the app.
- Added mail-4.svg, mail-5.svg, mail-6.svg, mail-7.svg, mail-8.svg, mail-9-plus.svg, mail-9.svg for various mail-related features. - Introduced puzzle.svg for a puzzle icon. - Included upload.svg for upload functionality.
ecnivtwelve
left a comment
There was a problem hiding this comment.
Bien dans l'ensemble, les icônes que je n'ai pas commenté me semblent prêtes à être ajoutées, et sinon c'est généralement des petits changements à faire. 👍🏽
J'attends l'avis de Tom pour la suite.
Si tu veux aller plus vite, tu peux ouvrir une PR avec les icônes prêtes (sans commentaire) pour les merge directement.
There was a problem hiding this comment.
Le gap entre les bulles est trop faible et les border des bulles trop élevé
There was a problem hiding this comment.
Manque peut être de corner smoothing ? @tom-things ?
There was a problem hiding this comment.
👍 Toujours du smoothing sur les icônes
There was a problem hiding this comment.
Pourrait reprendre le + existant avec une thickness plus faible au centre
There was a problem hiding this comment.
Les states Empty" et "Outline" sont incohérents avec Papicons
There was a problem hiding this comment.
Faut qu'on réfléchisse à l'intégration des nombres et du texte pour que ça reste lisible et cohérent avec le branding Papillon @tom-things
There was a problem hiding this comment.
Pour moi a part pour des edge case comme le PDF c'est pas nécessaire d'ajouter des chiffres
There was a problem hiding this comment.
Le pitch des pointillés pourraît être plus aligné avec l'espace avec le cercle ?
There was a problem hiding this comment.
On est pas dans la DA de Papicons ici, pas hyper pertinent
There was a problem hiding this comment.
C'est surtout que Papicons n'a pas de state remplissage et outline... C'est toujours rempli
There was a problem hiding this comment.
Y'a un folder qui arrive, donc celui-ci se fera écraser
There was a problem hiding this comment.
Bien, mais je trouve que le quart dans l'oeil ajoute une surcharge inutile
There was a problem hiding this comment.
👍 et le corner radius est trop faible sur les côtés
There was a problem hiding this comment.
Comme pour mail et PDF, faudra revoir l'intégration des chiffres
There was a problem hiding this comment.
👍 pour moi faudrait garder l'icône cloche sans les chiffres, pas nécessaire/pertinent dans un icon pack
There was a problem hiding this comment.
Comme pour bell et PDF, faudra revoir l'intégration des chiffres
tom-things
left a comment
There was a problem hiding this comment.
Certaines bonnes idées, mais pour la plupart des ajustements sont nécessaires pour que ça rentre dans la charte des Papicons, trop souvent des outlines trop fins ou des problèmes de lisibilité
There was a problem hiding this comment.
👍 et le corner radius est trop faible sur les côtés
There was a problem hiding this comment.
👍 pour moi faudrait garder l'icône cloche sans les chiffres, pas nécessaire/pertinent dans un icon pack
There was a problem hiding this comment.
C'est surtout que Papicons n'a pas de state remplissage et outline... C'est toujours rempli
There was a problem hiding this comment.
On est pas dans la DA de Papicons ici, pas hyper pertinent
There was a problem hiding this comment.
Peut-être bouger le minus un peu plus haut, sinon bien
There was a problem hiding this comment.
Bien, ce serait bien de supprimer la pupille directement
There was a problem hiding this comment.
La partie basse de l'engrenage pourrait-être supprimée
|
Dac besoin d'avoir accès au figma avec les originaux ? |
|
Oui ce serait bien ! |
|
dac |
|
Je ferme la PR ou pas @tom-things @ecnivtwelve |
Rules I followed :
#000000, without contouring (no stroke)Added :
history-clock-full.svghistory-clock.svgsidebar-empty.svgsidebar-full.svgtext-bubble-forum.svgplus-empty.svguser-plus.svguser-minus.svgand more...