Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#38538 closed defect (bug) (fixed)

Twenty Seventeen: Update page column setting

Reported by: melchoyce's profile melchoyce Owned by: laurelfulford's profile laurelfulford
Milestone: 4.7 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

Right now the page column setting is hidden when you have widgets in the Sidebar widget area. Since pages don't ever display the Sidebar widget area, the setting shouldn't be effected by it.

The description should be updated to:

You can opt to display all pages with a one column or two column layout. When the two column layout is assigned, the page title is in one column and content is in the other.

That does leave the question of archive pages. I think that if you don’t have widgets in the Sidebar widget area, the columns setting should probably also apply to archives — but if you do have widgets in the Sidebar widget area, the setting should be ignored on archive pages.

Attachments (2)

38538.0.diff (1.7 KB) - added by celloexpressions 8 years ago.
First pass for customizer.
38538.1.diff (5.5 KB) - added by laurelfulford 8 years ago.

Download all attachments as: .zip

Change History (6)

#1 @melchoyce
8 years ago

  • Owner set to celloexpressions
  • Status changed from new to assigned

@celloexpressions
8 years ago

First pass for customizer.

#2 @celloexpressions
8 years ago

  • Owner changed from celloexpressions to laurelfulford

38538.0.diff covers the customizer; archives need updates in the CSS to allow for a one-column option. I'm going to leave that part for @laurelfulford since I'm less familiar with that.

I ended up taking the pages part out of the description as well, as it's not really accurate. We may need to continue iterating here.

#3 @laurelfulford
8 years ago

  • Keywords has-patch added

In 38538.1.diff I added some styles to make sure the archives also apply the one column layout when that option is selected and there are no sidebar widgets assigned.

#4 @davidakennedy
8 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 39078:

Twenty Seventeen: Update page layout setting for all use cases

  • Makes sure the page layout setting also applies to pages, and is not completely hidden from them when a sidebar widget is present. Pages do not have a sidebar, so that was incorrect.
  • Updates description in Customizer for brevity and accuracy.
  • Applies setting to archive pages as well.
  • Updates CSS to reflect changes.

Props celloexpressions, laurelfulford.

Fixes #38538.

Note: See TracTickets for help on using tickets.