NREL / floorspace.js

Other
68 stars 36 forks source link

Usability improvement requests #305

Open fsudaman opened 6 years ago

fsudaman commented 6 years ago

As our engineers use this tool more, they have sent through these improvement requests:

Please review and see if they are appropriate to make your usability improvement list. Thanks

ljbrackney commented 6 years ago

I'm of two minds regarding confirmation on delete - at least for spaces. Sometimes I want to plow through a lot of deletions, and confirmations would be onerous. If this feature were to be added, I would want a preference setting for confirmations.

shorowit commented 6 years ago

You should be able to undo deletion of floors/spaces, I tend to feel like that is sufficient?

fsudaman commented 6 years ago

Undo should be sufficient, yes. The context for this was that one of our engineers, when first learning the software, accidentally deleted a floor (see image). For whatever reason he couldn’t undo the change. Perhaps the real issue is not being able to undo in some situations, and if so the deletion of those elements should prompt a confirmation.

floorspace deletion button

shorowit commented 6 years ago

If you come across a situation where undo is not available or does not work as intended, please tell us the steps to reproduce to fix it. I did some quick testing and found I was able to undo the deletion of a story, but there very well might be a bug in some situations.

I will open a separate ticket on the delete button issue, that should be improved.