-
Notifications
You must be signed in to change notification settings - Fork 28
Conversation
docs/DEV.md
Outdated
- */tests/* tests automatiques | ||
- */views/* fichiers ejs pour le serveur d'application | ||
|
||
Il y a deux points d'entrés à l'application front, cela produit 2 bundles qui communiquent par event ou par window. |
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.
*entrées
docs/DEV.md
Outdated
###CSS | ||
Le BEM est une convention de nommage orienté composant il rend la maintenance du css plus facile, le trade off est qu'il est un peu verbeux http://getbem.com/naming/ | ||
Penser à utiliser un fichier par "composant" | ||
Affin d'éviter la guerre des z-index il est plus pratiques de les rassembler dans un seul fichier (ici z_index.scss). cela garanti de ne pas avoir a gérer un z-index:9999 |
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.
*Afin
docs/DEV.md
Outdated
###CSS | ||
Le BEM est une convention de nommage orienté composant il rend la maintenance du css plus facile, le trade off est qu'il est un peu verbeux http://getbem.com/naming/ | ||
Penser à utiliser un fichier par "composant" | ||
Affin d'éviter la guerre des z-index il est plus pratiques de les rassembler dans un seul fichier (ici z_index.scss). cela garanti de ne pas avoir a gérer un z-index:9999 |
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.
*garantit
docs/DEV.md
Outdated
###CSS | ||
Le BEM est une convention de nommage orienté composant il rend la maintenance du css plus facile, le trade off est qu'il est un peu verbeux http://getbem.com/naming/ | ||
Penser à utiliser un fichier par "composant" | ||
Affin d'éviter la guerre des z-index il est plus pratiques de les rassembler dans un seul fichier (ici z_index.scss). cela garanti de ne pas avoir a gérer un z-index:9999 |
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.
à gérer
docs/DEV.md
Outdated
Affin d'éviter la guerre des z-index il est plus pratiques de les rassembler dans un seul fichier (ici z_index.scss). cela garanti de ne pas avoir a gérer un z-index:9999 | ||
|
||
### JS | ||
En général il est toujours possible si on fait un querySelector pour modifier le dom on risque de ne pas mettre à jours le state de l'app et de générer une inconsistance. |
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.
*jour
docs/DEV.md
Outdated
En général il est toujours possible si on fait un querySelector pour modifier le dom on risque de ne pas mettre à jours le state de l'app et de générer une inconsistance. | ||
|
||
## TODO | ||
Npm install a pour post script la génération les fontes il doit être possible de faire une vérification avant de faire l'installation systématique. |
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.
*fonts (fontes is for musculation as far as I know :p)
docs/DEV.md
Outdated
|
||
Les fichiers dots sont terribles, trop dissociés du code metier | ||
|
||
Il serait intéressant de revenir sur la dissociation etre les deux fichiers et de laisser webpack se charger du chargement lazy. Il faudrait laisser à scene.js le chargement de la lib mapbox (scene étant une interface pour cette lib tout le reste du projet est agnostique à mapboxgl). |
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.
*etre -> entre
docs/DEV.md
Outdated
Les fichiers dots sont terribles, trop dissociés du code metier | ||
|
||
Il serait intéressant de revenir sur la dissociation etre les deux fichiers et de laisser webpack se charger du chargement lazy. Il faudrait laisser à scene.js le chargement de la lib mapbox (scene étant une interface pour cette lib tout le reste du projet est agnostique à mapboxgl). | ||
ensuite intégrer scene dans app.js puis retirer tous les soft binding (fire / listen / window.) |
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.
*soft bindings
docs/DEV.md
Outdated
|
||
Il y a un ticket tech debt sur Jira | ||
|
||
Il est fastidieux de mettre à jours une icône par iconmoon, il doit être possible d'automatiser la tache. |
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.
*jour
No description provided.