theodore-norvell / PLAAY

Senior Design Project PLAAY (Programming Language for Adults And Youth)
2 stars 0 forks source link

Animator display errors #125

Closed WNewhook closed 6 years ago

WNewhook commented 6 years ago

Added better error display to animator.

theodore-norvell commented 6 years ago

The code looks good and seems to work well. However, I'm not keen on how this looks to the user. The way you did it, the error replaces the node where the error happens. This makes it harder for the programmer to see the problem. E.g. error-in-place I think it might be better if the error message were either at the top of the svg area like this error-at-top or under the offending node like this. error-below

theodore-norvell commented 6 years ago

This is much better now.

jillhancock commented 6 years ago

I just played around with it; looks really good.