Skip to content

Commit

Permalink
fix(stark-ui): properly size .stark-main-container on different scr…
Browse files Browse the repository at this point in the history
…een sizes

BREAKING CHANGES: the main container (`main.stark-main-container`) will now have a 100% width on some smaller (< 1100px) screen sizes.
  • Loading branch information
carlo-nomes committed Apr 29, 2019
1 parent 686c0f7 commit f8e4441
Showing 1 changed file with 3 additions and 15 deletions.
18 changes: 3 additions & 15 deletions packages/stark-ui/assets/styles/_base.scss
Original file line number Diff line number Diff line change
Expand Up @@ -21,25 +21,13 @@ body {
flex: 1 0 auto;
align-self: center;

max-width: 100%;
width: 100%;
max-width: $stark-max-content-width;

box-sizing: border-box;
padding: 64px 15px;
}
}
@media #{$tablet-query} {
.stark-main-container {
padding-left: 70px;
padding-right: 70px;
}
}

// We want to keep at least 70px horizontal padding for the main content, therefore we add 140px to the media query
@media screen and (min-width: 1100px) {
.stark-main-container {
align-self: center;
width: $stark-max-content-width;
}
}

.stark-loading {
margin: 16px auto;
Expand Down

0 comments on commit f8e4441

Please sign in to comment.