Opened 12 years ago
Closed 12 years ago
#29947 closed defect (bug) (fixed)
Reference to "Theme Customizer" in Customizer screen reader text
| Reported by: | celloexpressions | Owned by: | studionashvegas |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.1 |
| Component: | Customize | Version: | 4.0 |
| Severity: | minor | Keywords: | good-first-bug has-patch needs-testing commit |
| Cc: | Focuses: | accessibility |
Description
We're moving away from the term "Theme Customizer" in favor of just "Customizer". We missed an instance of "Theme Customizer" in the aria-label attribute of the #customize-info section at the top of the Customizer (which says "you are customizing..."); the "Theme" part should be removed.
Attachments (3)
Change History (10)
@
12 years ago
Use this patch instead - I missed the "instance of Theme Customizer" in the commented section.
#6
@
12 years ago
Awesome, 29947-customize-patch.2.diff takes care of all instances including bundled themes. Technically we should also always be capitalizing "Customizer" everywhere, although that's not as important.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
29947 - removed instance of "Theme" from "Theme Customizer"