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 display: flex #82

Closed
wants to merge 1 commit into from
Closed

add display: flex #82

wants to merge 1 commit into from

Conversation

jgillich
Copy link

I'm not sure how to name this one, since flex is obviously taken.

ref #13

@lukewestby
Copy link
Collaborator

this is the case for a couple other property keys/values as well, like top and bottom. can you take a look at https://github.com/rtfeldman/elm-css/blob/master/src/Css.elm#L521-L529 as a starting point for how this was handled for those?

@jgillich
Copy link
Author

I've looked into it, but this is a bit much for me right now; as mentioned in the original issue, I'm just getting started with Elm. Will revisit this in a couple of days/weeks.

@lukewestby
Copy link
Collaborator

sounds good! this is definitely a peculiarity of css that feels odd to represent in elm. anyone you can find on this repo or in the elm slack channel or elm-discuss mailing list would be happy to help out as you're working on it!

@rtfeldman
Copy link
Owner

I added displayFlex to implement this. Thanks!

@rtfeldman rtfeldman closed this Mar 5, 2016
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.

3 participants