Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Commit

Permalink
fix(layout): Firefox and flex layout with min-height
Browse files Browse the repository at this point in the history
* while Chrome has fixed this bug, Firefox still needs this workaround.

Fixes #7382.
  • Loading branch information
ThomasBurleson committed Mar 3, 2016
1 parent bfb5d5d commit ee3cab5
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions src/core/services/layout/layout.scss
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,9 @@
max-width: #{$value};
max-height: 100%;
box-sizing: border-box;

// Bug workaround for http://crbug.com/546034 - flex issues on Chrome 48
@if $i == 0 { min-width: 0; }
}

.layout-column > .#{$flexName}-#{$i * 5},
Expand All @@ -171,6 +174,10 @@
max-width: 100%;
max-height: #{$value};
box-sizing: border-box;

// Bug workaround for http://crbug.com/546034 - flex issues on Chrome 48
@if $i == 0 { min-height: 0; }

}
}

Expand All @@ -184,6 +191,9 @@
.layout-column, .layout#{$name}-column {
> .#{$flexName}-33 , > .#{$flexName}-33 { flex: 1 1 33.33%; max-width: 100%; max-height: 33.33%; box-sizing: border-box; }
> .#{$flexName}-66 , > .#{$flexName}-66 { flex: 1 1 66.66%; max-width: 100%; max-height: 66.66%; box-sizing: border-box; }

// Bug workaround for http://crbug.com/546034 - flex issues on Chrome 48
> .flex { min-height: 0; }
}
}

Expand Down

0 comments on commit ee3cab5

Please sign in to comment.