Opened 6 months ago
Closed 6 months ago
#22463 closed enhancement (fixed)
Gallery overlay controls disappearing
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Editor | Version: | 3.4 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: |
Description
If you've inserted a gallery into your post and you click the gallery placeholder in the Visual editor, the 'Edit Gallery' and 'Delete Gallery' buttons appear, but they disappear again after about 4 seconds. They should be persistent while the gallery placeholder is selected.
Attachments (2)
Change History (14)
comment:1
SergeyBiryukov — 6 months ago
- Version changed from trunk to 3.4
- Milestone changed from Awaiting Review to 3.5
Would be a good thing to fix. Should be a matter of removing the timeout (and probably just always showing them).
SergeyBiryukov — 6 months ago
comment:3
SergeyBiryukov — 6 months ago
- Keywords has-patch added
comment:4
koopersmith — 6 months ago
- Owner set to koopersmith
- Resolution set to fixed
- Status changed from new to closed
In 22652:
- Resolution fixed deleted
- Status changed from closed to reopened
This shows these buttons even when the new media modal is opened and as they have higher z-index, they are on top.
To truly hide the buttons when a gallery or an image is deselected, this would need to be looking at blur for the image which is a bit tricky in contenteditable.
- Resolution set to fixed
- Status changed from reopened to closed
In 22857:
Uh, my bad. Should have added a patch as we are in RC :(
If we are changing when the popup buttons are shown, perhaps can show them on mouseover (after short delay)? That will ensure hiding them on mouseout. Another option would be to always show them although that won't look very good on images.
- Resolution fixed deleted
- Status changed from closed to reopened
_hideButtons() uses tinymce.DOM.hide() and passes an array with element IDs to it, however if none of the elements exist on the page, it throws an error.
22463-2.patch fixes _hideButtons() by using Sizzle to pre-select the elements.
comment:10
nacin — 6 months ago
- Keywords commit added
comment:11
nacin — 6 months ago
Also tested by SergeyBiryukov.
comment:12
nacin — 6 months ago
- Resolution set to fixed
- Status changed from reopened to closed
In 22883:

Reproduced in 3.4.2 as well.