Skip to content
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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

epost
Copy link

@epost epost commented May 15, 2019

This enhancement was made by kettlebell: kettlebell@d6d96ca.

@epost
Copy link
Author

epost commented May 15, 2019

FYI @kettlebell, I made a PR out of your contribution.

@kettlebell
Copy link

Hi @epost, thanks for arranging the pull request. I've recently returned to my repo and updated it to use Spago and Halogen 5 (wasn't too bad as you can see here)

Also it'd be great to flesh out some more of the SVG properties.

@epost
Copy link
Author

epost commented May 16, 2019

@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?

@rskew
Copy link

rskew commented Aug 23, 2019

@epost I've started using this repo and I've added a couple of small things here #14 (it looks like I've duplicated the marker functionality, woops).

If you have another repo for merging various changes I'll happily create a PR there :)

@epost
Copy link
Author

epost commented Aug 23, 2019

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.

@rskew
Copy link

rskew commented Aug 24, 2019

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
Copy link
Owner

@epost @rskew - sorry for the long silence. As you guys can see, I'm no longer doing any purescript development so I don't have any personal use for this library. Would you prefer to be collaborators here, or alternatively I can replace it with a placeholder redirecting to another repo.

@rskew
Copy link

rskew commented Nov 11, 2019

@kwohlfahrt all good :) I'm up for whatever.

@srghma
Copy link

srghma commented Jul 31, 2020

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants