-
Notifications
You must be signed in to change notification settings - Fork 33
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add CSS font-size property (by kettlebell) #11
base: master
Are you sure you want to change the base?
Conversation
FYI @kettlebell, I made a PR out of your contribution. |
@kettlebell That's good to hear! I'm doing this in the hopes of converging on a repo that can be maintained, because we're using this at Statebox and I've added a fair number of features, including animation support. Here is a WIP document describing the current state of affairs: https://hackmd.io/lYvFcjb7STe2YzIet9sM7w?both. I've added the changes you just mentioned to the document. @kwohlfahrt Would you consider making some of us contributors, so we can collaborate on maintenance tasks? |
Hey @rskew, that is cool stuff! :D And a coincidence too; Sjoerd Visscher just started working on this yesterday at https://github.com/statebox/purescript-halogen-svg. Would be great if you could make a PR. |
Cheers @epost. I'll aim to make a PR soon, I'll have to update my parent project from Halogen 4 to 5 first. |
@kwohlfahrt all good :) I'm up for whatever. |
@kwohlfahrt hi, could you add me as a maintainer, I want to merge all currently open PRs and update to the latest halogen I want to use this repo in https://github.com/srghma/purescript-halogen-nextjs |
This enhancement was made by kettlebell: kettlebell@d6d96ca.