[wp-trac] [WordPress Trac] #34391: Harden panel/section UI code by removing contents from being logically nested (read: goodbye margin-top hacks)

WordPress Trac noreply at wordpress.org
Fri Sep 23 22:22:13 UTC 2016


#34391: Harden panel/section UI code by removing contents from being logically
nested (read: goodbye margin-top hacks)
-----------------------------+--------------------------------------------
 Reporter:  westonruter      |       Owner:  delawski
     Type:  defect (bug)     |      Status:  closed
 Priority:  normal           |   Milestone:  4.7
Component:  Customize        |     Version:  4.0
 Severity:  normal           |  Resolution:  fixed
 Keywords:  early has-patch  |     Focuses:  ui, accessibility, javascript
-----------------------------+--------------------------------------------
Changes (by westonruter):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"38648"]:
 {{{
 #!CommitTicketReference repository="" revision="38648"
 Customize: Re-architect and harden panel/section UI logic.

 Removes contents for sections and panels from being logically nested (in
 the DOM) in order to eliminate many issues related to using `margin-top`
 hacks. The element containing the link to expand the content element for
 panels and sections is now a sibling element to its content element: the
 content is removed from being nested at initialization. The content
 element is now available in a `contentContainer` property whereas the head
 element (containing the link to open the construct) is in a
 `headContainer` property. The existing `container` property is now a
 jQuery collection that contains both of these elements. Since the head
 element is no longer in an ancestor element to the `content` element, the
 `aria-owns` property is now used to maintain the relationship between the
 `headContainer` and the `contentContainer`. These changes are also
 accompanied by an improvement to the animation performance for the sliding
 panes.

 Props delawski, celloexpressions.
 Fixes #34391.
 Fixes #34344.
 Fixes #35947.
 }}}

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


More information about the wp-trac mailing list