-
Notifications
You must be signed in to change notification settings - Fork 250
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
flex: basis-size and full flex shorthand support #118
base: master
Are you sure you want to change the base?
Conversation
@@ -20,9 +20,6 @@ align-content() | |||
align-self() | |||
vendor('align-self', arguments, only: webkit official) | |||
|
|||
flex-basis() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved down to translated section.
anyone have comments on this? I haven't fooled with the flex-box stuff much yet, last time I checked it was pretty buggy in general |
Can I use FlexBox? (Dark green: obsolete and/or buggy, Light green: new) The obsolete model is relatively well supported (still buggy in some browsers). The new one isn't, but will be in the future and is the definitive way to go. But of course, I'd like to hear further opinions on this! |
I think you'll need to add |
Yup, you're right. I didn't add this file to the commit. Do you want to PR or shall I do it? |
I just created one at #126 |
New translations!
basis-size
now works with awidth
/height
hack.flex
now works as complete as possible. It supports all shorthand syntax versions defined in MDN and interprets<'flex-grow'>
and<'flex-basis'>
. We can't support<'flex-shrink'>
due to technical limitations of the old specifications.New properties, that are still untranslated:
align-content
algin-self
flex-shrink
flex-wrap
justify-content