Enhancement/components #40
Conversation
leihuayi
added
the
enhancement
There was a problem hiding this comment.
Ok pour moi, cependant j'ai des doutes sur les noms des composants. Comme ils sont déjà dans le dossier "components" ça me paraît redondant de rajouter "Component" à la fin. A moins que ce soit recommandé par la doc de Vue.js, ça me paraîtrait plus direct de les appeler "Header", "Result" et "Upload" tout simplement. Un composant par définition peut être réutilisable, donc hard-write dans le nom du header qu'il est pour l'instant que dans main n'est pas une bonne pratique selon moi car ça nous enlève la flexibilité de décider de le mettre dans d'autres pages (sinon on devra le renommer).
Est-ce que tu vas rajouter d'autres trucs à ta PR (page d'erreur, page d'infos, message sur le profil droit) ou ça sera dans une autre PR ?
| }, | ||
| autre_pistolet: { | ||
| displayLabel: "pistolet", | ||
| category: "B ou D" |
There was a problem hiding this comment.
on peut prendre la suggestion de Benjamin:
- label : pistolet divers
- catégorie: A, B ou D
There was a problem hiding this comment.
Merci je le fais
.gitignore
Outdated
| openrc.sh | ||
| backend/src/weights/* |
There was a problem hiding this comment.
pourquoi as-tu eu besoin de rajouter cette ligne ?
There was a problem hiding this comment.
j'avais les weights dans mon git status après un pull, je les enlève maintenant
|
Les noms des composants doivent être "multiwords" en Vue, donc on peut laisser comme ça. |
Divided previous unique component into: