Remove some styles

[trykickoff]

Remove some styles

[ashleynolan]

Definitely agree with removing the close button, address, and horizontal dl styling. Agree with keeping the unstyled list class and getting rid of the nav ul/ol selector too – can simply put the unstyled class on these elements if you need that styling that way.

Blockquote I’m not sure about whether we should remove – it’s quite a useful thing to have as it’s often used on sites. Any reason in particular you wanted to scrap?

[mrmartineau]

Thanks @ashleynolan, I felt that the blockquote is not useful enough to keep. It is one of those things that would change on almost every project & not integral to the framework. I don't mind keeping it, just wanted to gauge opinions.

I'll remove the others.

[mrmartineau]

After thinking about it some more, I'd like to remove some more of the button styles:

Large, Small, Pill, Block & Natural Buttons

I just don't think that they're used often enough. Perhaps the .btn--block could stay but I'm fairly certain the rest can be safely removed.

// Button Modifiers
// --------------------------------------------------
// These all follow the same sizing rules as above; text is 1em, space around it
// Large
.btn--large {
    @include font-size($type-large);
    line-height: 3;
    height: 3em;
    padding-right: 2em;
    padding-left: 2em;
    border-radius: 6px;
}

// Small
.btn--small {
    @include font-size($type-small);
    line-height: 1.5;
    height: 1.5em;
    padding-right: .75em;
    padding-left: .75em;
    border-radius: 3px;
}

.btn--pill {
    padding-left: 1.5em;
    padding-right: 1.5em;
    border-radius: 100em;
}

// Block button
// -------------------------
.btn--block {
    display: block;
    width: 100%;
    padding-left: 0;
    padding-right: 0;

    // Vertically space out multiple block buttons
    // same ad .btn--block + .btn--block
    & + & {
        margin-top: 10px;
    }
}


// Specificity overrides
input[type="submit"],
input[type="reset"],
input[type="button"] {
    &.btn--block {
        width: 100%;
    }
}

// Make the button inherit sizing from its parent.
.btn--natural {
    vertical-align: baseline;
    font-size: inherit;
    line-height: inherit;
    height: auto;
    padding-right: .5em;
    padding-left: .5em;
}

What do you think?

[mrmartineau]

I removed .btn--large, .btn--small, .btn--pill & .btn--natural but kept .btn--block