Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21404 closed enhancement (wontfix)

Twenty Twelve: "Only" use Customizer for Theme Settings

Reported by: obenland's profile obenland Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Currently we have an entire Theme Settings page for one checkbox in Twenty Twelve.

After reading Otto's second post on the Theme Customizer I'd be very intrigued to not offer a settings page but only use the Customizer to - well - customize the Theme. For the purpose of providing an implementation best practice for Theme Options we'd still have Twenty Eleven. Though after reading the article, I consider considering the Customizer as best practice. :)

What do you say?

Attachments (3)

21404.diff (10.4 KB) - added by obenland 12 years ago.
Possible implementation
21404.2.diff (10.4 KB) - added by obenland 12 years ago.
Web font default true
21404.3.diff (7.1 KB) - added by obenland 12 years ago.
Keeps Theme Options page for versions prior to 3.4

Download all attachments as: .zip

Change History (14)

#1 @helenyhou
12 years ago

Customizer would need to be made accessible first: #21283

#2 @scribu
12 years ago

  • Cc scribu added

@obenland
12 years ago

Possible implementation

#3 follow-up: @obenland
12 years ago

Option would not be available for WordPress versions prior to 3.4.

#4 in reply to: ↑ 3 ; follow-up: @DrewAPicture
12 years ago

  • Cc xoodrew@… added

Replying to obenland:

Option would not be available for WordPress versions prior to 3.4.

That's a good point. I still think this would be a great way to pimp using the Customizer for theme options. Can we do a back-compat fallback that adds the theme options page for pre-3.4?

#5 in reply to: ↑ 4 @obenland
12 years ago

Replying to DrewAPicture:

Can we do a back-compat fallback that adds the theme options page for pre-3.4?

If we'd keep using options instead of theme_mod all the way (for forward compatibility), I think we could.

Just for my info: Is not using a class for the Theme Options ... an option?

#6 @sabreuse
12 years ago

  • Cc sabreuse@… added

#8 follow-up: @Mamaduka
12 years ago

Nice idea, but as I remember from this week's dev chat Twenty Twelve compatibility is to 3.1, so I think we should leave back-comp for theme options.

I like implementing Theme Options as a class, it make code cleaner and prefixing all the things easier.

@obenland
12 years ago

Web font default true

#9 @obenland
12 years ago

It would be 3.1 compatible as in: not break.

There's a lot of possibilities:

  • Not have the web font option available until 3.4 with Open Sans disabled until Customizer available. (21404.diff)
  • Not have the web font option available until 3.4 with Open Sans enabled by default until Customizer available. This would keep the example of using a web font available for all versions. (21404.2.diff)
  • Have pre 3.4 Theme Option only, then Customizer only. Might lead to some duplicate code, though.
  • Leave as is.

@obenland
12 years ago

Keeps Theme Options page for versions prior to 3.4

#10 in reply to: ↑ 8 @lancewillett
12 years ago

Replying to Mamaduka:

Nice idea, but as I remember from this week's dev chat Twenty Twelve compatibility is to 3.1, so I think we should leave back-comp for theme options.

Agreed.

I like implementing Theme Options as a class, it make code cleaner and prefixing all the things easier.

Yes, that's why we did it.

I vote to leave this as-is.

#11 @lancewillett
12 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Discussed in IRC today to leave current behavior in place.

Note: See TracTickets for help on using tickets.