-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Page template + Sidebar #29
Conversation
alex-418
commented
Sep 28, 2017
@bdolor yarn.lock 430 deletions 🤔 no bueno ? |
</aside> | ||
@endif | ||
</div> | ||
<div class="wrap container container-fluid " role="document"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
container
+ container-fluid
is redundant. one or the other. (container-fluid
)
@endif | ||
</div> | ||
<div class="wrap container container-fluid " role="document"> | ||
<div class="content clearfix row-fluid"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.row-fluid
exists in bs3 only. bs4 it is row
. what purpose does .clearfix
serve?
</div> | ||
<div class="wrap container container-fluid " role="document"> | ||
<div class="content clearfix row-fluid"> | ||
<main class="col-sm-8 clearfix"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.clearfix
necessary? .main
is declared in bs4 - why was it was taken out?