Opened 11 years ago
Closed 11 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: |
|
Owned by: |
|
|---|---|---|---|
| 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
@
11 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
debounceSearchon both theinputandchangeevents. 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.AvailableMenuItemsPanelViewseems to fix it, but it may be there for a reason I'm not seeing.