Repository navigation
inline form doc example #23452
Description
Hi
https://getbootstrap.com/docs/4.0/components/forms/
inline form
the provided example seems to me confusing
it's work well at smallest screen and when the space is enough to display in one line BUT on screen size who need two line we dont have y margin between form elements...
here is a code pen .. with a possible correction if necessary ?
Activity
mattxwang commentedon Aug 16, 2017
Hey @jipexu , I tried reproducing your bug and wasn't able to replicate the issue. On my browser, each element in
form-inlinestacks properly with a vertical margin on a small viewport. Here's a screenshot of what I mean:In addition, I took a look at your codepen, and I wasn't able to find a difference between the two different examples (other than a bottom margin on the second inline form, but that exists on any viewport size).
Both of these screenshots were taken on Chrome Version 60.0.3112.90.
Can you upload some screenshots of what problem you're encountering? Also, what browser are you using?
jipexu commentedon Aug 16, 2017
@malsf21 thanks but i have this behavior on Safari, FF, and Chrome (all latest version) on Mac Sierra ?...
??
jipexu commentedon Aug 16, 2017
jipexu commentedon Aug 16, 2017
jipexu commentedon Aug 16, 2017
i dont really said it's bug i just say the html must provide some "y" class ... and in the Doc the provided class are not (enough or correct) ...
mattxwang commentedon Aug 16, 2017
Ah, I see what you're saying, that does look like a problem. If you already have a fix, you can submit a Pull Request against
v4-dev. I can also help you do that if you want.jipexu commentedon Aug 16, 2017
Thanks ! No i have no idea if we must just a PR to correct the code of the doc Or if we have to go further in css ?
mattxwang commentedon Aug 17, 2017
I mean it's up to you, I'm not sure how easy (or necessary) a CSS fix even is. A PR to update the doc is probably enough for now, but you can also make a small note about the problem you faced in the docs.
jipexu commentedon Aug 17, 2017
yep probably adjust the code doc by a pr
@mdo or team please confirm if usefull or not ?? if yes i will make pr
mdo commentedon Oct 2, 2017
A PR would be great @jipexu.
jipexu commentedon Oct 23, 2017
@mdo oups sorry i missed this i will do it ! (because i see it's same in the new doc)