I have never done this before so my apologies if I'm doing this wrong. Basically, I wanted to add the plus icon used in the onboarding flow. I followed the steps in the README to update and download the fonts, but it seems like their output format has changed (tabs to spaces, chars ordered by code) which resulted in noisy diffs in this PR. Let me know if there's anything I should fix before merging.
This LGTM, but it's unfortunate you went through all this effort 😛 because nowadays we just embed the icon in the HTML which is much simpler, see here and here. Happy to have this merged though, just for next time you might save your self some ceremony.
craigspaeth
commented
6 years ago
I have never done this before so my apologies if I'm doing this wrong. Basically, I wanted to add the plus icon used in the onboarding flow. I followed the steps in the README to update and download the fonts, but it seems like their output format has changed (tabs to spaces, chars ordered by code) which resulted in noisy diffs in this PR. Let me know if there's anything I should fix before merging.