Skip to content
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

spacing problem between dropdowns on dropdown docs #31841

Closed
wants to merge 6 commits into from

Conversation

mznet
Copy link

@mznet mznet commented Oct 5, 2020

image

I've found that a spacing problem between dropdowns on docs. This problem is caused by left margin even though line breaks. To fix problem, I removed left margin between dropdowns on xs size and applied the margin on over sm size.

You can check the issue on Sizing section of dropdown page. Check it on xs size screen. And this issue is happened to v4 as well as v5.

Preview: https://deploy-preview-31841--twbs-bootstrap.netlify.app/docs/5.0/components/dropdowns/#sizing

@mznet mznet requested a review from a team as a code owner October 5, 2020 14:52
@XhmikosR XhmikosR added this to Inbox in v5.0.0-alpha3 via automation Oct 5, 2020
@XhmikosR XhmikosR added this to Inbox in v4.5.3 via automation Oct 5, 2020
mdo
mdo previously requested changes Oct 5, 2020
site/content/docs/5.0/components/dropdowns.md Outdated Show resolved Hide resolved
v5.0.0-alpha3 automation moved this from Inbox to Review Oct 5, 2020
@mznet mznet requested a review from mdo October 7, 2020 00:07
Copy link
Member

@MartijnCuppens MartijnCuppens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @mznet!

v5.0.0-alpha3 automation moved this from Review to Approved Oct 9, 2020
mdo added a commit that referenced this pull request Oct 9, 2020
- Drop the included .btn-toolbar, no need
- Split large and small button examples

Closes #31841
@mdo
Copy link
Member

mdo commented Oct 9, 2020

Replacing this with #31861—while reviewing, I realized we could split those examples and avoid unnecessary changes to our components CSS (which negatively impact all many other btn-groups in other snippets).

@mdo mdo closed this Oct 9, 2020
@mdo mdo removed this from Approved in v5.0.0-alpha3 Oct 9, 2020
@mdo mdo removed this from Inbox in v4.5.3 Oct 9, 2020
XhmikosR added a commit that referenced this pull request Oct 13, 2020
- Drop the included .btn-toolbar, no need
- Split large and small button examples

Closes #31841

Co-authored-by: XhmikosR <xhmikosr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants