Opened 15 years ago
Closed 15 years ago
#18045 closed enhancement (fixed)
Replace tabled-based settings form markup with do_settings_sections() in Twenty Eleven
| Reported by: | chipbennett | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.3 |
| Component: | Bundled Theme | Version: | 3.2 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
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
@
15 years ago
- Milestone Awaiting Review → 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
@
15 years ago
- Priority normal → high
Crap. I'm not sure how we missed theme_options_render_page().
That needs to be fixed.
#4
@
15 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
@
15 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
implements do_settings_sections(), add_settings_section(), and add_settings_field() to replace hard-coded, table-based settings page markup