Opened 11 years ago
Closed 11 years ago
#33630 closed defect (bug) (fixed)
Delete menu - similar strings and HTML
| Reported by: | pavelevap | Owned by: | ocean90 |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.4 |
| Component: | Menus | Version: | 4.4 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
- Menu in Customizer
String: "Delete menu"
HTML: Button
https://core.trac.wordpress.org/browser/tags/4.3/src/wp-includes/class-wp-customize-control.php#L1585
- Menu page
String: "Delete Menu" (both capital letters)
HTML: Link
https://core.trac.wordpress.org/browser/tags/4.3/src/wp-admin/nav-menus.php#L813
1) I only wanted to merge same strings, but I found other issues.
2) Also HTML should be the same (button)?
3) {{ data.menu_name }} as screen reader text does not print anything.
4) For localization it should be string like Delete menu <span class="screen-reader-text">%s</span>?
Change History (2)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In 35695: