Opened 13 years ago
Closed 13 years ago
#26647 closed defect (bug) (fixed)
Similar strings in Twenty Fourteen
| Reported by: | pavelevap | Owned by: | lancewillett |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Bundled Theme | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
I am not sure if there is any hidden meaning for both strings, but I did not find anything. Could we merge these long strings?
Attachments (1)
Change History (7)
#2
@
13 years ago
Yes, exactly, maybe the whole $wp_customize->add_section part could be removed? Not sure about that...
#5
@
13 years ago
- Keywords has-patch added; needs-patch removed
I think we need to add the section in both places, since we're adding controls to it in both places. I believe this was done to allow the Featured Content file to be easily used in other themes, with the Twenty Fourteen-specific option defined separately in the theme.
Patch standardizes both places to use the shorter string, from twentyfourteen_customize_register().
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
From my testing, the one in
Featured_Content::customize_register()is never displayed in the UI, the one intwentyfourteen_customize_register()overrides it.