Skip to content

Commit

Permalink
Removed prefixes from flexbox and other mixins
Browse files Browse the repository at this point in the history
It was made clear to me that I should leave vendor prefixing to post-processing tools like autoprefixer. I will leave the mixins in for sheer usefulness, but I'll otherwise recommend that you add a post-processing step to your CSS!
  • Loading branch information
bhalash committed Jun 28, 2015
1 parent 3bcbc1c commit afab14f
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 11 deletions.
21 changes: 11 additions & 10 deletions _mixins.scss
Expand Up @@ -307,32 +307,33 @@

@mixin flex-parent($direction: row, $justify-content: null, $wrap: null, $align-items: null) {
// Generate flex parent rules.
@include prefix-value(display, flex, webkit);
@include prefix(flex-direction, $direction, webkit);

disiplay: flex;
flext-direction: $direction;

@if ($justify-content) {
@include prefix(justify-content, $justify-content, webkit);
justify-content: $justify-content;
}

@if ($wrap) {
@include prefix(flex-wrap, $wrap, webkit);
flex-wrap: $wrap;
}

@if ($align-items) {
@include prefix(align-items, $align-items, webkit);
align-items: $align-items;
}
}

@mixin flex-child($flex: 1, $align-self: null, $order: null) {
// Generate flex children rules.
@include prefix(flex, $flex, webkit);
flex: $flex;

@if ($align-self) {
@include prefix(align-self, $align-self, webkit);
align-self: align-self;
}

@if ($order) {
@include prefix(order, $order, webkit);
order: $order;
}
}

Expand Down Expand Up @@ -377,8 +378,8 @@
}

@mixin columns($count, $gap) {
@include prefix(column-count, $count);
@include prefix(column-gap, $gap);
column-count: $count;
column-gap: $gap;
}

/**
Expand Down
2 changes: 1 addition & 1 deletion _partials.scss
Expand Up @@ -92,7 +92,7 @@

.noselect, %noselect {
cursor: default;
@include prefix(user-select, none);
user-select: none;
}

/**
Expand Down

0 comments on commit afab14f

Please sign in to comment.