-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Custom font #33
Custom font #33
Conversation
i've just spent 10 minutes on writing code and another 30 on trying to please the linter 馃槚 the worst thing is as soon as I fix the problem it demands to rewrite the very thing I've just fixed in another way... would you mind adding some instructions on the correct workflow in |
Actually, that code is good, but I did not describe the ticket well enough. I will merge it and create another ticket describing it a bit more in detail. |
@all-contributors please add @merelj for code |
I've put up a pull request to add @merelj! 馃帀 |
Thank you for accepting the PR and sorry I didn't exactly get it right. 馃槚 I would gladly refactor it at some point in the future (not sure as to when though) if no one beats me to it 馃憤 |
PR for issue Custom Font
1st commit contains all the actual changes (component code + story)
2nd commit is just a result of me runningCI linter failed with this commit so I removed it. Fails anyway, tho...npm run lint -- --fix
since pre-commit hook would fail otherwise 馃槙Eager to hear your feedback; from what I can gather I should also add an example in codesandbox, but I'm not sure how to do it since I'd have to use new version of this package 馃