Opened 4 years ago
Closed 3 years ago
#51877 closed defect (bug) (fixed)
Twenty Twenty: Wrong parameter name used in searchform.php
Reported by: | poena | Owned by: | sabernhardt |
---|---|---|---|
Milestone: | 5.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
In searchform.php, the parameter/argument that is used is named
$args['label']
, the correct name for use with get_search_form() is $args['aria_label']
.
See https://developer.wordpress.org/reference/functions/get_search_form/#parameters
Attachments (2)
Change History (9)
#1
@
4 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
@
4 years ago
Wrong parameter name used in searchform.php in Twenty Twenty changed $argslabel? to $argsaria_label?
@
4 years ago
editing argument to aria-label
in get_search_form()
functions for 3 files, and updating searchform.php to give a fallback for child theme templates that might use label
#3
@
4 years ago
- Keywords has-patch added; needs-patch removed
This was a little more complicated than I had thought it might be. In addition to updating the argument in 4 files, 51877.2.patch adds a fallback for child theme backward compatibility.
#4
@
4 years ago
- Milestone changed from Future Release to 5.8
- Owner set to sabernhardt
- Status changed from new to accepted
Note: See
TracTickets for help on using
tickets.
Would this be a good first bug?
In addition to replacing 'label' with 'aria_label', Twenty Twenty's searchform template still needs to accept 'label' as a valid parameter (if used in a child theme).