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

Add class which removes gutter between columns #13551

Closed
wants to merge 4 commits into from
Closed

Add class which removes gutter between columns #13551

wants to merge 4 commits into from

Conversation

kperwel
Copy link

@kperwel kperwel commented May 9, 2014

It adds ".no-gutter" class for rows, which removes columns gutter. I've used attribute selector to omit enlarging of grid.


.no-gutter
{
margin-left: 0 !important;
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think using !important is gonna fly.

Copy link
Author

Choose a reason for hiding this comment

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

Yeah, you're right. It's unnecessary.

@cvrebert cvrebert changed the title [FEATURE] Add class which removes gutter between columns Add class which removes gutter between columns May 9, 2014
// Remove gutter between columns in row

.no-gutter
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this brace on a new line?

@mdo
Copy link
Member

mdo commented May 10, 2014

We'll pass on adding this for now.

@mdo mdo closed this May 10, 2014
@gabskoro
Copy link

@mdo Can you explain why are you passing this for now? Do you have any other plans about removing gutter, or you don't want it at all?

@kperwel
Copy link
Author

kperwel commented May 24, 2014

This is just feature, new possibility. IMO quite useful. Some elements don't needs gutter, but should use grid system. Just new quite usefull idea. Some developers asking for solution to remove defult gutter for some elements.

@kperwel kperwel deleted the no-gutter-columns branch May 24, 2014 16:13
@kperwel kperwel restored the no-gutter-columns branch May 24, 2014 16:13
@twbs twbs locked and limited conversation to collaborators Feb 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants