[wp-trac] [WordPress Trac] #49288: Metabox holders missing their border and "Drag boxes here" text

WordPress Trac noreply at wordpress.org
Wed Jul 29 19:08:26 UTC 2020


#49288: Metabox holders missing their border and "Drag boxes here" text
-------------------------------------+-------------------------------------
 Reporter:  xkon                     |       Owner:  audrasjb
     Type:  defect (bug)             |      Status:  reopened
 Priority:  highest omg bbq          |   Milestone:  5.5
Component:  Administration           |     Version:
 Severity:  blocker                  |  Resolution:
 Keywords:  has-screenshots needs-   |     Focuses:  ui, accessibility,
  testing needs-backwards-           |  administration
  compatibility needs-patch          |
-------------------------------------+-------------------------------------

Comment (by azaozz):

 Replying to [comment:71 afercia]:
 > I'm not sure the CSS part is the main backwards compatibility concern

 Yes, looking more now, some of the js changes would need tweaking/fixing.

 > - Previously, the `empty-container` class set via JS was used only on
 the Dashboard and in the `#post-body` layout only for the `side-
 sortables`, only in the 2 columns layout.
 > ...
 > - The lack of a min-height is the reason why it was not possible any
 longer to move a postbox into the `advanced-sortables` area when empty:
 this was a bug, see comment:7

 Right. The drop areas on the old Edit Post screen are actually three.
 However the second and third are always shown together. If I remember
 correctly the reason the third is three is that postboxes added by plugins
 were put by default in it; at the bottom under the editor. Also think at
 some point (over 10 years ago) that area used to be shown separately, but
 now is always shown together with the second area. So when dragging of
 postboxes was added to that screen, the three areas were kept as back-
 compat, but the third was (mostly) disabled. The users could drag boxes
 out but there's no reason for them to add boxes back in it, as they would
 show and work in exactly the same way as when in the second area. In that
 terms the changes here can either maintain the previous behavior or make
 the third area "droppable" again. This won't change how the screen works.

 > - Regardless, this `visible == 1` has an impact also on the `#post-body`
 layout so I think the best option for now is to just restore it.

 Sounds good. Yes, ideally it should have been implemented better.

 > Seems to me the best way to address backwards compatibility. If there's
 consensus, I can turn this into a patch.

 Yes, I think all the outlined reasons and changes make sense (sorry I
 can't be of more help atm, having some health issues).

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/49288#comment:72>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list