covidtrends icon indicating copy to clipboard operation
covidtrends copied to clipboard

Add per-capita option

Open jwosty opened this issue 4 years ago • 6 comments

Addresses #1

jwosty avatar Mar 29 '20 18:03 jwosty

This is ready for a review. I've added a dropdown where you can choose between cases "absolute" and "per capita".

jwosty avatar Mar 29 '20 22:03 jwosty

I was going to add a pro capita toggle! I'm happy you're already working on it! It would be great!

pleasehelpalex avatar Apr 10 '20 01:04 pleasehelpalex

FYI, you should read the comment @aatishb wrote on this topic some time ago: https://github.com/aatishb/covidtrends/issues/30#issuecomment-607927252

nweldev avatar Apr 15 '20 20:04 nweldev

Just want to remind folks of our code of conduct, which I’m excerpting here.

Be respectful. Disagreement is not an opportunity to attack someone else's thoughts or opinions. Although views may differ, remember to approach every situation with patience and care.

aatishb avatar Apr 16 '20 14:04 aatishb

@mrelg this was just an FYI. Nothing more than "hey BTW, there is a discussion about that, maybe it could be useful to read it". I personally run this branch locally on a daily bases, as it's useful to me alongside the version on the master. Yet, I understand and respect @aatishb work and opinion. Anyone who doesn't is free to run their own fork or better, to contribute to the discussion respectfully.

nweldev avatar Apr 16 '20 14:04 nweldev

@mrelg We have deleted your comments for being off-topic and not living up to our code of conduct.

Just to be clear, all of these comments are being posted on a public thread. As far as we can tell, @noelmace’s comment was not addressed to you but to the whole audience on this thread. You may inadvertently be getting email notifications from this comment thread, which you are free to unsubscribe from.

We will also be deleting any further off-topic comments. Just as a heads up, repeated code of conduct violations will result in being blocked from this repository.

– @aatishb & @mustpax

aatishb avatar Apr 16 '20 17:04 aatishb