Opened 23 months ago
Closed 18 months ago
#18045 closed enhancement (fixed)
Replace tabled-based settings form markup with do_settings_sections() in Twenty Eleven
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.3 |
| Component: | Bundled Theme | Version: | 3.2 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | chip@…, ian@… |
Description
Currently, Twenty Eleven uses a hard-coded, table-based markup for its Theme Options page. Patch replaces this markup with do_settings_sections(), corresponding call to add_settings_section(), and calls to add_settings_field() for each Theme Option.
One other, minor change: changed theme_options_render_page() function name to twentyeleven_theme_options_render_page(), to correspond to function prefix best practices.
I would like to add in more inline documentation to this file, in general; but want to get at least a blessed first-pass on the code changes while I work on improving the documentation.
Attachments (3)
Change History (19)
chipbennett — 23 months ago
- Milestone changed from Awaiting Review to 3.2.1
Moving to 3.2.1. Don't expect it then, but would like to roll it out in a Twenty Eleven point release.
- Priority changed from normal to high
Crap. I'm not sure how we missed theme_options_render_page().
That needs to be fixed.
comment:4
chipbennett — 23 months ago
Well let me know if there's anything else that needs to be fixed/addressed/polished up in the patch, and I'll get it turned back around.
Or, if you would prefer, I could pull the theme_options_render_page() fix out into its own patch, for expediting?
- Keywords needs-testing removed
Attached patch:
- Moves add_settings_section and add_settings_field calls into twentyeleven_theme_options_init().
- Removes unneeded $default_options variables from each of the settings fields callbacks.
- Provides a way to hide the section title, in do_settings_sections.
- Milestone changed from 3.2.1 to 3.2.2
Soaking the rest in trunk and punting this to 3.2.2.
comment:10
nacin — 23 months ago
- Priority changed from high to normal
comment:11
nacin — 23 months ago
- Keywords commit added
comment:12
iandstewart — 22 months ago
- Cc ian@… added
comment:13
nacin — 22 months ago
In [18455]:
comment:14
nacin — 22 months ago
In [18458]:
comment:15
nacin — 22 months ago
- Keywords fixed-major added
comment:16
nacin — 18 months ago
- Keywords fixed-major removed
- Milestone changed from 3.2.2 to 3.3
- Resolution set to fixed
- Status changed from new to closed

implements do_settings_sections(), add_settings_section(), and add_settings_field() to replace hard-coded, table-based settings page markup