Closed aschleck closed 4 years ago
This is lovely. Question: do you prefer to launch as soon as we put in the geo stuff, or do you want to make it more perfect. :)
There will always be feature catchup, not sure if we will be feature In-parity. Examples:
Really up to you. I’d be supportive of what you decide.
April Schleck notifications@github.com於 2020年5月8日 週五,下午11:43寫道:
- search box
- highlighted the additional numbers you suggested in the top bar
- tweak the graphs into something more useful
- fixed the mobile layout
[image: image] https://user-images.githubusercontent.com/115766/81466297-69354c80-9185-11ea-8ce8-f3eace7ae09c.png [image: image] https://user-images.githubusercontent.com/115766/81466337-b2859c00-9185-11ea-9545-7bf4ec52246c.png
You can view, comment on, or merge this pull request online at:
https://github.com/xunhuang/covid-19/pull/117 Commit Summary
- Add a search box, tweak the graphs, fix the mobile layout, and probably some other nonsense.
File Changes
- M website/package.json https://github.com/xunhuang/covid-19/pull/117/files#diff-07f3a4647460e74f9514d8def137963a (1)
- M website/src/App.js https://github.com/xunhuang/covid-19/pull/117/files#diff-0946eb2e3b6b3a75a32049b27295c879 (7)
- M website/src/Footer.js https://github.com/xunhuang/covid-19/pull/117/files#diff-bd0156e96dffdf7b750066fbab722231 (3)
- A website/src/components/chrome/SearchInput.js https://github.com/xunhuang/covid-19/pull/117/files#diff-bde8c18a4251cf7e6b851ea41daeb600 (141)
- M website/src/components/tables/DivisionTable.js https://github.com/xunhuang/covid-19/pull/117/files#diff-bb011ad9deeeeb189cb662fd8f878e80 (2)
- M website/src/components/tables/SortableTable.js https://github.com/xunhuang/covid-19/pull/117/files#diff-3d5d511162636fdf3769d22533e38cd1 (14)
- M website/src/models/Earth.js https://github.com/xunhuang/covid-19/pull/117/files#diff-0288bbd770525f9349caf5b221556dd3 (169)
- A website/src/models/SearchIndexComponent.js https://github.com/xunhuang/covid-19/pull/117/files#diff-032131411ffab8b6ee1e0c0d1c8c6d89 (63)
- M website/src/models/World.js https://github.com/xunhuang/covid-19/pull/117/files#diff-d9b61b9d543e6ae7cb832e1ab43dccea (12)
- M website/src/pages/PageRegion.js https://github.com/xunhuang/covid-19/pull/117/files#diff-a028aa41c5ed5488a10f089ed14457a6 (147)
Patch Links:
- https://github.com/xunhuang/covid-19/pull/117.patch
- https://github.com/xunhuang/covid-19/pull/117.diff
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/xunhuang/covid-19/pull/117, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFSGF5CTYM2K4NRIBVWHIDRQT3SDANCNFSM4M4VOWLQ .
Ahhhh too much responsibility :)
I guess launching earlier is better than launching later. I have 0 motivation to touch maps, so definitely happy to not wait for that. I do have an aspirational goal to enable comparing places on the same graph, but that might take me a long time so waiting for it would probably be a mistake.
By the way I looked around for geo location stuff, have you tried https://ip-api.com/? No idea if it's good, but it's apparently completely free for non-profit. I don't have the API keys for the geo IPs and some of the stuff makes me nervous (API key #1, API key #2, API key #3...) and makes me want to not touch it. So not sure how much work I can do on integrating that stuff. But the good/bad news with country data is it's really rare to have anything other than country level, so just taking the country code from ipdata or ip-api or whatever is probably perfect. I was planning on putting a crosshair icon (equivalent to "Find My Location") into the right side of the search box at some point, but that doesn't seem like much of a blocker.
Projections would be fun and I did some thinking about how to support the envelope in the UI. I don't really want to write the code to compute them though (I guess that would go into world_data_gen.js?) I think if you or someone else wanted to tackle computing them it'd be worth waiting for it, but otherwise I feel pretty meh.
Other things missing relative the main site:
Anyway yeah, I guess let's do the geo stuff and see how it goes?
Oh of course metros are missing too. But we have so little detailed country data that there's no point in metros outside the US
And links to public health departments are missing
hey, lets move this conversation to a github issue instead. the PR keeps closing. :)
On Sat, May 9, 2020 at 1:17 AM April Schleck notifications@github.com wrote:
Oh of course metros are missing too. But we have so little detailed country data that there's no point in metros outside the US
And links to public health departments are missing
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/xunhuang/covid-19/pull/117#issuecomment-626127311, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFSGF3QHJ4AN7RV7Z2CMLDRQUGRFANCNFSM4M4VOWLQ .
https://github.com/xunhuang/covid-19/issues/121
On Sat, May 9, 2020 at 10:52 AM Xun Wilson Huang xhuang@gmail.com wrote:
hey, lets move this conversation to a github issue instead. the PR keeps closing. :)
On Sat, May 9, 2020 at 1:17 AM April Schleck notifications@github.com wrote:
Oh of course metros are missing too. But we have so little detailed country data that there's no point in metros outside the US
And links to public health departments are missing
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/xunhuang/covid-19/pull/117#issuecomment-626127311, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFSGF3QHJ4AN7RV7Z2CMLDRQUGRFANCNFSM4M4VOWLQ .