[wp-trac] [WordPress Trac] #31476: Semantic elements for non-link links: /wp-admin/includes/widgets.php

WordPress Trac noreply at wordpress.org
Wed May 3 02:23:32 UTC 2017


#31476: Semantic elements for non-link links: /wp-admin/includes/widgets.php
-------------------------------------+-------------------------------------
 Reporter:  afercia                  |       Owner:  afercia
     Type:  defect (bug)             |      Status:  reopened
 Priority:  normal                   |   Milestone:  4.8
Component:  Widgets                  |     Version:  4.1
 Severity:  normal                   |  Resolution:
 Keywords:  semantic-buttons needs-  |     Focuses:  ui, accessibility,
  patch                              |  javascript
-------------------------------------+-------------------------------------
Changes (by westonruter):

 * keywords:  has-patch semantic-buttons commit => semantic-buttons needs-
     patch
 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 r40480 introduced a regression in a widget control's contents when it is
 expanding, namely the widget is not taking the full
 width while animating. See [attachment:animated-widget-contents-
 width.mov].

 This seems to mostly fix the issue:

 {{{#!diff
 --- src/wp-admin/css/customize-widgets.css
 +++ src/wp-admin/css/customize-widgets.css
 @@ -87,6 +87,8 @@
         padding: 1px 10px 10px 10px;
         border-top: none;
         line-height: 16px;
 +       box-sizing: border-box;
 +       width: 100%;
  }

  .customize-control-widget_form.expanded .widget-action .toggle-
 indicator:before {
 }}}

 I say ''mostly'' because there is one other oddity in that there is a 1px
 border at the top of the `.widget-inside` while animating and then it
 collapses once the animation finishes.

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


More information about the wp-trac mailing list