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

Container padding uses per-breakpoint gutter sizes, fixes #19304. #20973

Merged
merged 1 commit into from
Oct 25, 2016

Conversation

pixelbandito
Copy link
Contributor

@pixelbandito pixelbandito commented Oct 22, 2016

Hoping this is what was intended by issue #19304. I'm re-using the per-breakpoint setting for gutters as the padding base for containers.

@mdo mdo added this to the v4.0.0-alpha.6 milestone Oct 23, 2016
@pixelbandito
Copy link
Contributor Author

pixelbandito commented Oct 24, 2016

@mdo I closed this PR because the Savage check seemed to be stalled. I didn't see that you'd already added it to the milestone.

I opened a different PR instead (#20984) to try to trigger a new check, hoping the stalled test would resolve itself.

Sorry for messing up the process. Would you like me to close the other PR and reopen this one?

For future reference: is it normal for Savage checks to stay in a pending state for multiple days?

@bardiharborow
Copy link
Member

bardiharborow commented Oct 24, 2016

Savage is always a little weird. Just ignore it unless it fails. I think reopen this one.

@pixelbandito pixelbandito reopened this Oct 24, 2016
@mdo
Copy link
Member

mdo commented Oct 24, 2016

I closed this PR because the Savage check seemed to be stalled. I didn't see that you'd already added it to the milestone.

Yeah, unsure what's up with that. Maybe @cvrebert can peep it?

@mdo mdo merged commit a3bc24a into twbs:v4-dev Oct 25, 2016
@mdo mdo mentioned this pull request Oct 25, 2016
@mdo mdo mentioned this pull request Jun 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants