Add flex-support for Col component #169#175
Add flex-support for Col component #169#175eddywashere merged 4 commits intoreactstrap:masterfrom ronnyhaase:flex-support-col
Conversation
|
@ronnyhaase nice work! I meant to finalize the options. After thinking about it, I don't think |
|
I agree regarding Also allow me to ask if you prefer a rebase for removing I will make the changes and add documentation later on. Thank you! |
|
Re: Boolean, I was originally cool with it. I'm a little hesitant to commit to it. Just doing the auto seems safer to maintain and is explicit. Not super strict but I like to rebase. Either way I squash into a single commit when merging pr in github. |
|
Hej! So I removed "flex" as valid option, and bool as valid type for size of config object. But unfortunately these examples are not working, since the docs use the Bootstrap release from NPM which has flex disabled by default. How should I proceed from here? (I joined the Slack channel, feel free to ping me there) |
|
that's a really good question. react-helmet might support this. If not, we might need to hack in a separate html template for flexbox pages. If you need this out asap. I can merge it without the docs. let me know what you think. |
Hi!
As requested in issue #169 I implemented support for dropping the size for the Col component, respectively flexbox support.
I added support for all suggestions by @eddywashere, meaning a value can be dropped at all, or set to "flex" and "auto", like:
If the object's size shall accept boolean as well, and both "flex" and "auto" should be supported may needs some discussion?
Corresponding unit tests were added as well.
Looking forward to your feedback
Ronny