Replace restaurants with food options and update components styles #7
Conversation
ameliedefrance
commented
Apr 16, 2024
•
ameliedefrance
commented
- Replace restaurants with food options
- Rename components
- Update theme & components styling
ameliedefrance
changed the title
LedruRomane
force-pushed
the
refacto-vite-react-ts
branch
from
4a75272 to
b1dde0f
Compare
| @@ -1,20 +1,17 @@ | |||
| import '@assets/app.scss' | |||
| import AppLayout from '@app/layouts/AppLayout.tsx'; | |||
| import AppLayout from '@app/layouts/app-layout.tsx'; | |||
There was a problem hiding this comment.
C'est une nouvelle pratique d'ajouter des tirets et retirer les majuscules ? :o Je découvre
There was a problem hiding this comment.
Ah bah alors aucune idée, c'est comme ça sur Rigby et je sais pas si ça vient de nos pratiques ou des leurs ! Là bas les noms de fichiers sont en kebab case et les composants en pascal case, j'ai fait pareil 🙈
There was a problem hiding this comment.
Ah non, ce sont des pratiques propres à Rigby (une préférence de Guillaume par rapport à des soucis de casse sur Windows qu'il a pu rencontrer). On s'est alignés sur le projet, mais je suis pour conserver du PascalCase pour les composants et leurs noms de fichier chez nous 😃
(le kebab-case peut-être conservé mais plutôt pour des dossiers, pas des fichiers)
There was a problem hiding this comment.
Team Pascal et pas kebab, c'est noté 🌮
|
|
||
| // TODO: Unused component, keep for future version ? |
There was a problem hiding this comment.
Oui, au cas où... M'enfin ça partira dans l'été si on n'en a pas l'utilité 👍
