Add sidebar to fallback frontpage. #43
Conversation
| &-sidebar { | ||
| border: 1px solid #dae1e7; | ||
| border-radius: .25rem; | ||
| padding: 15px; |
There was a problem hiding this comment.
Replace this with a call to config($spacing, 'default') (for 20px padding).
| <div class="frontpage-fallback-minutes"> | ||
| <?php | ||
| $minutes->the_post(); | ||
| <div class="main-inner"> |
There was a problem hiding this comment.
.main-inner is the wrapper for the whole page content, this should not be repeated here.
| &-sidebar { | ||
| border: 1px solid #dae1e7; | ||
| border-radius: .25rem; | ||
| padding: 15px; |
There was a problem hiding this comment.
Replace this with a call to config($spacing, 'default') (for 20px padding).
| <?php | ||
| $minutes->the_post(); | ||
| <div class="main-inner"> | ||
| <div class="frontpage-fallback-minutes-main"> |
There was a problem hiding this comment.
This should have a wrapper with .frontpage-fallback-minutes
There was a problem hiding this comment.
Or replace .main-inner with .frontpage-fallback-minutes
|
I think we have to add the changes proposed in #41 in this step as well. |
|
It might be best to take a step back and streamline the content so that the front page only has one wrapper instead of two and only the contents change if there is no meetup. |
This pull request adds a sidebar to the fallback frontpage template (and also the latest commits from the master branch).