Opened 8 years ago
Closed 8 years ago
#38953 closed defect (bug) (fixed)
Customize Menus: clicking outside of the available menu items panel does not close the panel
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.7.4 | Priority: | normal |
Severity: | normal | Version: | 4.6.1 |
Component: | Customize | Keywords: | has-patch commit fixed-major |
Focuses: | javascript | Cc: |
Description
When the available menu items panel is open, clicking anywhere outside of the panel except for on the menu-item-delete buttons should (and used to) close the panel. In particular, clicking on the preview and clicking on an added menu item to expand its options currently do not close the available menu items panel, but clicking elsewhere in the customize pane works.
Verified that this behavior exists in 4.6.1, so it was a regression some time before 4.7.
Attachments (3)
Change History (24)
#4
@
8 years ago
- Keywords commit added
38953.2.diff looks good and works in my testing. Still not sure when/how this broke.
#7
@
8 years ago
- Keywords needs-patch added; has-patch commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
This change broke the bulk-delete functionality. The available menu items panel can't be closed if the click is on the item-delete button within the menu item handle.
Again, the specific functionality that this ticket looks to restore previously worked, and I specifically remember adding the check for the delete button too. Not sure what happened to it.
This ticket was mentioned in Slack in #core-customize by melchoyce. View the logs.
8 years ago
#10
@
8 years ago
- Focuses javascript added
- Keywords good-first-bug added
Since this was committed to trunk previously, two things are now required:
- Amend the trunk fix to not close the panel when the delete buttons are activated. (good-first-bug, needs a patch)
- Back-port both commits to the 4.7 branch.
This ticket was mentioned in Slack in #core by swissspidy. View the logs.
8 years ago
@
8 years ago
Fixes menu delete button and prevents available menu items from closing when removing an item
This ticket was mentioned in Slack in #core by swissspidy. View the logs.
8 years ago
#16
@
8 years ago
@westonruter
I tested 38953.diff and it works as expected --
Before the patch, every time i click the red X to remove a menu item, the side panel closes:
After the patch I can remove multiple items and the side panel stays open:
This patch also fixes the delete menu link not working issue:
#18
@
8 years ago
- Keywords commit added; good-first-bug removed
- Owner set to westonruter
- Status changed from assigned to accepted
Added pointer-events none to preview iframe when the available menu items panel is open. Then closes the panel on #customize-preview & .menu-item-handle click