We have implemented two AJAX calls for a few features. AJAX calls might take a while depending on the connection speed to respond. So the user has to wait for the AJAX response after they perform some action. At this point, if we don't show the user that we are acting upon their actions in the background, they might think that the application is not working.
We have two such actions:
1) While creating a new report, upon a change in the state, the list of the locations is changed through AJAX. As per this PR, until the request completes, the select box will become disable and a user will see the notification in the label that we are fetching locations for the state.
2) On Heatmaps page, when a user clicks on a location, we fetch the data through AJAX. As per this PR, the user will see an infowindow mentioning that we are working on fetching the reports for the location. Design of the infowindow is not great. It will require some effort to fix it.
Othe changes with this PR:
Fixed failing tests
Few code refactors
Clicking anywhere on the map will close the currently opened infowindow
We have implemented two AJAX calls for a few features. AJAX calls might take a while depending on the connection speed to respond. So the user has to wait for the AJAX response after they perform some action. At this point, if we don't show the user that we are acting upon their actions in the background, they might think that the application is not working.
We have two such actions: 1) While creating a new report, upon a change in the state, the list of the locations is changed through AJAX. As per this PR, until the request completes, the select box will become disable and a user will see the notification in the label that we are fetching locations for the state.
2) On Heatmaps page, when a user clicks on a location, we fetch the data through AJAX. As per this PR, the user will see an infowindow mentioning that we are working on fetching the reports for the location. Design of the infowindow is not great. It will require some effort to fix it.
Othe changes with this PR: