transitPath: fix nodes list when path has no segments #789
Conversation
The `segments` data attribute may be empty if the path was imported from gtfs. Undefined exceptions were thrown and the interface was blank when deleting nodes from the nodes list.
| ) | ||
| } | ||
| /> | ||
| )} |
There was a problem hiding this comment.
C'est une question de préférence mais je ne suis pas certain que cette version "formattée" est plus lisible que la version précédente qui tient sur deux lignes
There was a problem hiding this comment.
C'est discutable. La version formatée fitte dans une ligne de fenêtre vscode de développemtn (~120 caractères max) alors que la version sur 1 ligne oblige à scroller vers la droite, d'où la longueur limite des lignes. Et c'est pas moi qui formatte cela comme ça pour le fun, c'est en roulant yarn format. Sinon, chaque fois qu'on roule la commande, il y aura des fichiers modifiés. Le formatage, par définition, c'est "opinionated". On prend une décision un jour (qu'on revisite parfois, genre quand on change la version des linters/prettier) et après on vit avec, même si des fois ça a l'air weird. C'est le prix à payer pour avoir un code style commun.
There was a problem hiding this comment.
Je vois!
C'est vrai que ça évite de devoir scroller pour voir toute la ligne (sauf si on active word wrapping)
Gardons cette version formattée alors :-)
The
segmentsdata attribute may be empty if the path was imported from gtfs. Undefined exceptions were thrown and the interface was blank when deleting nodes from the nodes list.Also run
yarn formaton the code