Page – Full width

First, very nice theme and thanks for sharing it with us!

Just an observation and no way to say this without sounding opinionated but the template “Page – Fullwidth” implies that the_content is not contained within a grid container and is actually the full width of the browser however the_content is still contained within a 900px grid and is not actually full width as many would expect.

It should be named “Page – No Sidebar” because it’s simply a page without right/left sidebars but still is not truly full-width and could keep people from using this theme.

Simple css fix of course but many will not know how to address this on their own, I suggest splitting into two templates.

Current “Page – Fullwidth” template:

.page-full-contain {
    max-width: 900px;
    margin: 0 auto;
}

(should rename template to “Page – No Sidebar”)

A new “Page – Full Width” template could use:

.page-full-width {
    max-width: 100%;
    margin: 0 auto;
}

This will also allow your theme to be used with popular page builders, sliders, etc where a need for full width sections is desired.

• This topic was modified 7 years, 10 months ago by pingram.