[wp-trac] [WordPress Trac] #49288: Metabox holders missing their border and "Drag boxes here" text
WordPress Trac
noreply at wordpress.org
Fri Jul 3 19:26:16 UTC 2020
#49288: Metabox holders missing their border and "Drag boxes here" text
-------------------------------------+-------------------------------------
Reporter: xkon | Owner: audrasjb
Type: defect (bug) | Status: reviewing
Priority: normal | Milestone: 5.5
Component: Administration | Version:
Severity: normal | Resolution:
Keywords: has-patch has- | Focuses: ui, accessibility,
screenshots | administration
-------------------------------------+-------------------------------------
Changes (by azaozz):
* keywords: has-patch has-screenshots commit => has-patch has-screenshots
Comment:
Replying to [comment:33 afercia]:
> - please double check the removed check for `visible == 1` as I'm not
sure I understand ~~why it was there in the first place~~ [edit] whether
it's still used (it was introduced in [20272])
Hmm, can't remember (it was 8 years ago!). Seems it had something to do
with hiding the outlines for empty drop areas on the Dashboard when there
is only one "box" available for dragging. Then it doesn't make sense to
show other areas as there's nothing to move there.
49288.7.diff seems to work well on the Dashboard, but doesn't work on the
old Edit Post screen (need Classic Editor to test). When there are no
boxes under the editor, or whgen all of them are (visibly) hidden, you
cannot drag a box from the side to under the editor.
Seem caused by
{{{
#normal-sortables.empty-container, #post-body #advanced-sortables.empty-
container {
height: auto;
...
}}}
For Sortables the drop area has to have width and height before you start
to drag, or there's nowhere to drop. Removing the above CSS seem to fix
it.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/49288#comment:34>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list