-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Improve IE compatibility for flex-1 utility #549
Comments
This sounds like Flexbug #6. Speaking of Flexbugs however, the above flex definition you're now using might cause Flexbug #4, which you could solve by using this: flex: 1 1 0%; There's also luisrudge/postcss-flexbugs-fixes which fixes both #4 and #6. You could just wack that into your PostCSS config after Tailwind. |
Ah, thank you for the info, I always forget about that awesome repo. Would there be any downside to having our |
Would totally accept a PR for this change 👍🏻
…On Mon, Sep 10, 2018 at 12:56 PM Scott Bedard ***@***.***> wrote:
Ah, thank you for the info, I always forget about that awesome repo. Would
there be any downside to having our flex-1 utility written this way out
of the box? It seems like a bit of a gotcha to use the shorthand, and then
leaving it up to the user to apply this fix. If the maintainers oppose this
change though, no worries, go ahead and close this issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#549 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEH3bCJU-anwnONkBGc7bH3tcKShxqnQks5uZpmqgaJpZM4Wf-Ku>
.
|
IE has trouble understanding our
flex-1
utility, perhaps we should avoid using theflex: 1
shorthand and be more explicit. After some digging, I wound up on this stackoverflow page and am now overwriting myflex-1
class with the following...FWIW, Edge seems to handle the existing utility just fine.
The text was updated successfully, but these errors were encountered: