acteng / atip

Active Travel Infrastructure Platform
https://acteng.github.io/atip/
Apache License 2.0
22 stars 4 forks source link

Handling schema changes mid-edit #254

Closed robinlovelace-ate closed 1 year ago

robinlovelace-ate commented 1 year ago

I know the schema dropdown is experimental but still think it's worth exploring:

dabreegster commented 1 year ago

Re-thinking the schema dropdown, maybe it better belongs as a big modal that lists the other schemas, describes their purpose, and makes it clear the entire page/app is about to change substantially. Or maybe we don't even have a dropdown, and direct people for each use case using a URL we manually give them right now.

Worth notifying users that their edits will re-appear

It could be. Similar to how "clear all" today has a confirmation modal, we could use this as a place to let people know what's going on.

Can objects edited with one schema be imported into a session that is using a different schema

No, by design, because the attributes per object are different. No known use case for doing this yet.

robinlovelace-ate commented 1 year ago

Or maybe we don't even have a dropdown, and direct people for each use case using a URL we manually give them right now.

That one sounds better to me at this stage: each use case is so domain specific it deserves its own URL and I can rarely/ever imagine users needing to switch mid edits. So big +1 to encoding schema in URL not dropdown from me.

robinlovelace-ate commented 1 year ago

That would fix the issue for me. I think there may still need to be documentation and defensive programming that catches situations in which people accidentally try to upload edits made in one schema to another but that's probably capture in another issue. Another thing I like about the URL-based schema selection: simpler UX and hides away that dropdown that can could cause confusion.

dabreegster commented 1 year ago

Should anything in the sidebar emphasize what mode we're in? What if somebody has used both planning and criticals mode, and is confused about what they're looking at? Screenshot from 2023-06-27 16-27-36

robinlovelace-ate commented 1 year ago

Yes there should be a heading, e.g. "Scheme design mode", "Development planning mode", "Critical issue mode", "Experimental mode" where we can test solutions to #231