Opened 7 years ago
Closed 7 years ago
#41803 closed defect (bug) (fixed)
themes.php search form doesn't work when submitted
Reported by: | boonebgorges | Owned by: | afercia |
---|---|---|---|
Milestone: | 4.9 | Priority: | normal |
Severity: | normal | Version: | 4.8 |
Component: | Themes | Keywords: | has-patch |
Focuses: | javascript | Cc: |
Description
wp-admin/themes.php has a live search powered by Backbone. But if you enter a term and hit 'Enter', forcing a proper form submit, you're led back to themes.php *without* the search
URL param being filled in.
Could we do one of the following?
- Prevent form submission here, either by preventDefault() on submit, or by rejecting the Enter key
- Add
name="search"
to the live search inputattributes
, so that form submission works normally
Previously: #34881 - cc @afercia
Attachments (1)
Change History (7)
#2
@
7 years ago
@afercia [40572] added a <form>
element around the input. In 4.7, there was no form, so hitting Enter didn't do anything.
#3
@
7 years ago
- Keywords needs-patch added; 2nd-opinion removed
I guess because in 4.7 there wasn't a <form>
element wrapping the search field, see [40572] 😬
#4
@
7 years ago
- Focuses javascript added; administration removed
- Keywords has-patch added; needs-patch removed
- Milestone changed from Awaiting Review to 4.9
@boonebgorges 41803.diff should fix it, mind a quick test? Thanks!
Note: See
TracTickets for help on using
tickets.
@boonebgorges I can't reproduce on previous 4.7 where the form submission seems correctly prevented. Maybe something missed with recent changes in 4.8.