buidl-labs / foamviz

Central repository containing data visualizations over FOAM's TCR Data
http://foamviz.surge.sh/
4 stars 2 forks source link

Viz1 UX fixes #100

Closed prastut closed 4 years ago

prastut commented 4 years ago

Breathing space in control panel: image

Currently, elements are too tightly placed together.

dhruv10 commented 4 years ago

Fixed in #102

dhruv10 commented 4 years ago

Two UX issues:

  1. Jittery animation of tower objects I was able to reproduce the bug only when 2 - 4 instances of web app were open in different tabs simultaneously. It depends on CPU usage probably. Needs to be rigorously tested.

  2. Delayed rendering of towers The API response is very slow. Changing the 10k limit of objects (a query parameter in API) to 1k reduced fetching time from 11s (approx) to 8s (approx) that too on average not always. Though further reducing in limit showed not much improvement and also we cannot guard it further otherwise incomplete/ wrong information would be perceived by the FOAMViz users. Here's the API for testing bbox-details-nyc

dhruv10 commented 4 years ago

Delayed rendering of towers cause of slow API response

Here are screenshots from both Postman and Insomnia: Check the API response time The exact endpoint is this

image

image

prastut commented 4 years ago
dhruv10 commented 4 years ago

No loading shown while fetching points from FOAM. When viz shows Delhi's map -> there is a slight delay to fetch points and then render towers on the screen. That delay from a UX perspective was earlier signified by a loader.

True, this is needed as fetching takes a very significant time now

dhruv10 commented 4 years ago

106