Closed milafrerichs closed 8 years ago
the following needs to be included:
Updated the popup a bit.
Styling would be next
This is great! Can you add per-capita income? The demographics should should match those in the corner buttons of the screenshot at https://github.com/codeforamerica/resident-web-use-research/issues/5.
I’m seeing Uncaught ReferenceError: L is not defined(anonymous function) @ lib.js:485
in Chrome 43.
Updated the library loading to not use a protocol. Jekit uses HTTPS by default and this caused a "Mixed Content" error. Should be fixed now: Staging Preview
Jekit’s not going to work here; Census reporter requests are blocked as insecure content.
did a bit of polishing:
Staging on AWS there are a few things to work on:
These are looking excellent.
Add more infos to the popup
Just added a few more infos into the popup. The infos that are available in the geojson.
Screenshot
Discussion
What should we include? I would suggest the infos that are in the CensurReporter and the amounts from the API. Would suggest to use underscore. And find the data with it (filter, where). I would suggested underscore anyway since it will improve the performance and readability for the loops and so forth (issue #13 )
What do you think?