Opened 10 years ago
Closed 10 years ago
#32677 closed defect (bug) (fixed)
Customizer Menus: Accordion toggle in available menu items panel fails on first attempt after a search
Reported by: | tywayne | Owned by: | ocean90 |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | 4.3 |
Component: | Customize | Keywords: | needs-patch |
Focuses: | Cc: |
Description
Steps to reproduce:
- Click Add Items when editing a menu
- Use the search field, doesn't matter if it returns results or not.
- Attempt to toggle one of the accordion items below.
gif of process here: https://cloudup.com/c2w8vM4sheu
Attachments (1)
Change History (4)
#2
@
10 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 4.3
Related comment by @aferica: https://wordpress.slack.com/archives/core-customize/p1434409353000585
Note: See
TracTickets for help on using
tickets.
After looking further, it appears this is happening because we're triggering
debounceSearch
on both theinput
andchange
events. Is there a reason we need to trigger a search on both of those events? The change event is firing when the input loses focus, thus opening the search results again.Removing the change event in
api.Menus.AvailableMenuItemsPanelView
seems to fix it, but it may be there for a reason I'm not seeing.