Update comp calculator to show location as modifier instead of including it in the benchmark

[ryansydnor]

Changes

I thought it was a little bit confusing that the location multiplier was already included in the "benchmark", despite a tooltip stating that the benchmark was based on San Francisco. I tried to make that clearer by always showing the San Francisco benchmark followed by a separate line for the location modifier.

Side Note: I found the number of options in the dropdown confusing. Especially because most of them don't actually impact the result (i.e. all 33 of the everywhere else options in the US have the same location modifier). I could see only having options for major cities with unique location modifiers, and then a single "Everywhere else" option. It looks like we're already doing this in other countries. Was there a reason for adding so many unique "Everywhere else" options in the US?

[vercel[bot]]

@ryansydnor is attempting to deploy a commit to the PostHog Team on Vercel.

A member of the Team first needs to authorize it.

[ryansydnor]

@corywatilo, could you take a peek at this?

(@jurajmajerik told me you would be the right person to tag in our recent interview)

[MarconLP]

Hey @ryansydnor, this was intentionally implemented in this PR, since the SF location benchmark itself isn't really relevant to other locations.