-
Notifications
You must be signed in to change notification settings - Fork 0
Design #3
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
base: 07-routing
Are you sure you want to change the base?
Design #3
Conversation
package.json
Outdated
"@emotion/react": "^11.14.0", | ||
"@emotion/styled": "^11.14.0", | ||
"@mui/icons-material": "^6.4.8", | ||
"@mui/material": "^6.4.8", | ||
"devicon": "^2.16.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Est-ce que c'est vraiment utile d'ajouter toutes ces libs qu'il va falloir maintenir ?
Je vois par exemple que @mui/icons-material
tu ne t'en sers qu'une fois..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alors pour les @emotion
je peux les virer, j'ai suivi bêtement l'install mais sans ca marche aussi
pour l'icon, moi je suis d'avis de la laisser, MUI marche avec son propre truc d'icon, on pourrait a la rigueur utilise le github.svg si tu penses que c'est mieux, perso pour les deux options j'y vois pas d'inconvénient
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je trouve qu'ajouter toute une lib pour au final n'utiliser qu'une seule icône provenant de cette lib ça ne vaut pas le coup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all right je vais la tejjjjj
Related Slides :
https://knplabs.slides.com/knpteam/react-typescript-version?token=gMZoow4s#/108