.container width overriden #48959

Closed
opened 2016-07-28 01:25:42 +02:00 by nucularJohn · 4 comments

Examples: From sizes 768px to 1100px

https://cloud.blender.org/blog/ - Affects page-body and footer.
https://cloud.blender.org/services - Affects footer.


Observed behaviour:

On resize: _base.sass:35 (see below) overrides all @media's width rules.

.container {
    min-width: 1100px;
}

So from width 768px to 1100px it stays as 1100px. An unexpected behavior I suppose.


Proposed solution:

The easiest way would be to just remove .container's min-width rule.

But usually on a "Mobile First" approach the rules you have on _utils.sass:167 (see below) would be the default rules and then you would resize with @media accordingly. I would suggest doing that.

// _utils.sass:167

@media (max-width: 767px)
.container {
    max-width: 100%;
    min-width: auto;
    padding-left: 0;
    padding-right: 0;
}
**Examples:** *From sizes 768px to 1100px* https://cloud.blender.org/blog/ - Affects page-body and footer. https://cloud.blender.org/services - Affects footer. --- **Observed behaviour:** On resize: `_base.sass:35` (see below) overrides all `@media`'s width rules. ``` .container { min-width: 1100px; } ``` So from width 768px to 1100px it stays as 1100px. An unexpected behavior I suppose. --- **Proposed solution:** The easiest way would be to just remove `.container`'s `min-width` rule. But usually on a "Mobile First" approach the rules you have on `_utils.sass:167` (see below) would be the default rules and then you would resize with `@media` accordingly. I would suggest doing that. ``` // _utils.sass:167 @media (max-width: 767px) .container { max-width: 100%; min-width: auto; padding-left: 0; padding-right: 0; } ```
Author

Changed status to: 'Open'

Changed status to: 'Open'
Author

Added subscriber: @nucularJohn

Added subscriber: @nucularJohn

Added subscribers: @dr.sybren, @fsiddi

Added subscribers: @dr.sybren, @fsiddi
Pablo Vazquez was assigned by Francesco Siddi 2016-07-29 11:57:53 +02:00

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: archive/pillar#48959
No description provided.