WRI-Cities / static-GTFS-manager

GUI interface for creating, editing, exporting of static GTFS data for a public transit authority
GNU General Public License v3.0
147 stars 46 forks source link

Default Route Sequence delete sequence #146

Open lightsprint09 opened 5 years ago

lightsprint09 commented 5 years ago

I just forgot to enter the Route in the Default Route Sequence screen. When saving it warned me to set a route. After setting it I lost all my data. Could be improved by setting and switching routes at any time

answerquest commented 5 years ago

@lightsprint09 Yikes! I remember how that feels.
The program flow is set up such that when you change to another route, whatever sequence is found gets loaded. So the switching routes part may not work out, because how is the program supposed to know if you want to load the new route's sequence or you want to keep what you've made.

I'm thinking of a prevention step instead : Disable the user from adding any stop unless they have chosen a route. Will work on that.. thanks for bringing this to my attention.

Another measure that might be useful (thinking aloud here) : set a global changesMade flag in the javascript. On any change done to a sequence, make it True. On saving, set to False. If user changes to another route while the flag is True, then do a popup confirmation dialog warning about data loss and asking to save things first.

answerquest commented 5 years ago

I also want to make the tabular data copy-paste-able to excel at some point. That way the user can also do bulk data-entry. User feedback helps to bring things up in priority list, so +1 this if you want that feature!