Twenty Seventeen: front page section placeholders toggle with theme options section but are visible initially

[celloexpressions]

The front page section placeholders were intended to only be visible when the theme options section was expanded, so that the preview is an accurate representation of the front end of the site. When the theme options section is expanded, the placeholders are shown along with borders on each section, to clarify where each option is applied on the site and where additional content can be added.

Currently in trunk, the placeholders are visible when the customizer loads, then properly toggle with the theme options section's exxpanded state. They should be hidden when the customizer first loads. One could argue that the current behavior improves discoverability, but it is more likely to cause annoyance when extra section placeholders continue to show up after initial theme setup, and starter content reduces the need for placeholders showing initially.

Not milestoning yet as it needs a patch, but this should probably be in 4.7.

[sstoqnov]

Hide front section panels on load

[sstoqnov]

I think that it's ok to hide them on load using twentyseventeen style.css

[helen]

This works for initial load and toggling in/out of theme options; however, when clearing a section (i.e. setting it to no page), the placeholder does not then appear. Should it? I'm a bit torn.

[sstoqnov]

hide twentyseventeen theme option placeholders on load

[sstoqnov]

Agree, then they should be hidden using javascript, so when clearing section they will be visible again.

[davidakennedy]

I agree too to that @helen. Thanks @sstoqnov for the updated patch! I tested this and it works well. This should be good to go.

[davidakennedy]

In 39417:

Twenty Seventeen: Hide front section panels on page load of Customizer.

Props sstoqnov.

Fixes #38951.

[helen]

In 39421:

Twenty Seventeen: Hide front section panels on page load of Customizer.

Props sstoqnov.

Fixes #38951 for the 4.7 branch.