Opened 13 years ago
Closed 13 years ago
#18045 closed enhancement (fixed)
Replace tabled-based settings form markup with do_settings_sections() in Twenty Eleven
Reported by: | chipbennett | Owned by: | |
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | normal | Version: | 3.2 |
Component: | Bundled Theme | Keywords: | has-patch commit |
Focuses: | Cc: |
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)
#1
@
13 years 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.
#3
@
13 years ago
- Priority changed from normal to high
Crap. I'm not sure how we missed theme_options_render_page().
That needs to be fixed.
#4
@
13 years 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?
#5
@
13 years ago
- 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.
#8
@
13 years ago
- Milestone changed from 3.2.1 to 3.2.2
Soaking the rest in trunk and punting this to 3.2.2.
implements do_settings_sections(), add_settings_section(), and add_settings_field() to replace hard-coded, table-based settings page markup