nestauk / arloesiadur_industry_dataviz Goto Github PK

Check the total business values...

At present, all files are called from the local filesystem. Some of these (for example jquery) would be better served from a CDN to take advantage of user caching.

This is great! Only a bit of polish left:

In the map
Need to explain what is the meaning of "Score" in the map tooltips (is this local economic complexity?)

In the bubble-chart:

-Number of businesses is a % (I assume). do *100 and add (%). Same with business growth (I assume it is a percentage too?)

-Add a legend for colours and sizes.

-The current palette looks nice but I am a bit concerned that it suggests qualitative differences (red vs green vs blue) rather than a gradient in the probability distribution. I think that red vs blue with grey in 0.5, and a black edge-colour to make sure that sectors in the middle of the distribution are visible might work better but this is one of those try and see what works situations.

-Round up all numbers to 2 decimals.

-Indicate that it is possible to use click and drag to zoom in different areas of the bubble chart.

-Add legend at the bottom including links to ONS, blog and GitHub.

And I think that's it - great work!

At the moment, all the data stored locally are loaded from separate files. This isn't efficient, as each occurs individually. The data, which are stored as JS variables, can be moved to a single file.

Need a Leaflet control for showing info on area when hovered. Example http://leafletjs.com/examples/choropleth.html.

When a user clicks an area fire the areaChanged event

Users need to be able to select an area by some means - either through a dropdown, a text search or some other functionality.

The chart is showing probability, so the xaxis would be better set at 0.5. Similarly, we want to differentiate on the Y axis to create quadrants.

When loaded no area is selected on the map, although the areas dropdown displays 'City of London'' and the chart displays the City of London data. The map should draw with the relevant LAD selected and should be fit to the bounds of that LAD.

The cursor changes to a hand icon whenever it's over the map. This should be disabled, as clicking has no functionality.

The current colorbar isn't great for representing the data due to the shades of grey in the mid-range.

Currently circle size is arbitrary. Add key if possible.

Need to explain what is the meaning of "Score" in the map tooltips (is this local economic complexity?)

When the chart is moused over, the plotly mode bar overlays it, meaning some tooltip information is behind the mode bar layer.

The mode bar can be configured using Plotly's confguration options https://plot.ly/javascript/configuration-options/.

Currently circle diameter is based only on the currently selected areas data - this is not representative due to influence of data etc. It would be better to use a value that is a rate of the local economy.