search

aclu-cash-bail-team / aclu-cash-bail

Code powering ACLU Pennsylvania's Broken Rules interactive
https://www.brokenrulespa.org
2 stars 0 forks source link

Merged Scatterplots #41

Closed marioaks closed 3 years ago

marioaks commented 3 years ago

1) Merges diverging scatterplot branches. I went ahead and removed the changes I made to Scatterplot and County Point and instead sorted the array of CountyPoints. As long as new scatterplots aren't introduced or changed, this should be ok.

2) Adds in the new Tooltip module & merges in the mobile tooltip styling defined in the scatter-plot branch

3) Serves table & scatterplot index files at the same time with parcel v2. Should be easy to add in a maps index file as well if it's convenient.

alexwgraves commented 3 years ago

This encapsulates all of the changes in #37, #38, and #39, right? If so can you close those PRs?

marioaks commented 3 years ago

This encapsulates all of the changes in #37, #38, and #39, right? If so can you close those PRs?

Closed #38 and #39. Added 2 comments to #37. Feel free to close that one once you see them.

alexwgraves commented 3 years ago

Closes #36