mitodl / rapid-response-xblock

a django app plug-in for edx-platform
BSD 3-Clause "New" or "Revised" License
0 stars 2 forks source link

Change general error handler to show inline message instead of overlay #66

Closed gsidebo closed 6 years ago

gsidebo commented 6 years ago

Related to #64 and #65

Issue #58 was created to show a user a general error message in a block-spanning overlay when one of our AJAX requests failed with a non-200 status. While working on #63, it made sense to show the user a less intrusive error message when a polling request timed out:

ss 2018-07-24 at 17 00 05

After discussing with @Ferdi, it's probably better UX to show general error messages in the same way instead of obscuring the entire rapid response content window with an overlay. The open/close button and the timer would be disabled, and the open problem spinner would be hidden in this general error state.