-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
When flex
is used as a display
value it should be autoprefixed
#257
Comments
Display options aren't whitelisted. It should just work, no? Sent from my phone
|
I see the value get added sometime but it's not prefixed and it doesn't seem to get added consistently. |
If you can quickly confirm that the only issue is not auto-prefixing (which is not currently part of what velocity does for the display property), then I'll add that feature in. |
Kay, looks like I lied about the inconsistency. Seems like the only issue is not autoprefixing. |
flex
should be a supported display
valueflex
is used as a display
value it should be autoprefixed
Okay. This will be fixed in the upcoming release. Ty. |
Thanks! |
wait. this was never a prob to begin with. are you sure the browser you're testing on actually supports flex as you intend it to? see http://caniuse.com/#search=flex |
oh nvm this shit is wack |
I didn't want to say wack... but it's pretty wack. |
Please test, confirm it works, and get back to me. Thank you so much! |
Yep, works great. You're rad. |
+1 for this just working |
Dropped support for the easeBack and easeElastic easings. They are rarely used, look stupid, and take up a bunch of lines. Sped up bezier curve easing performance. Thanks, @gre. The display option now gets browser-prefixed if necessary. This inherently adds cross-browser support for the ‘flex’ value. Closes #257. Fixed bug where slideDown/up left excess inline styles on elements. Closes #260. cc @kpeatt, @scalvert. Allow passing an empty string to `display` to remove the inline style from the element. Closes #184. Shimmed version of jQuery’s $.dequeue() now accepts a raw DOM element *set* instead of just a single element. You can now stop custom queues individually. Closes #262.
Right now, the
display
function takes three supported values:auto
,none
, andblock
. It would be great if Velocity would take aflex
option as well and autoprefix it to support older browsers.The text was updated successfully, but these errors were encountered: