[wp-trac] [WordPress Trac] #22839: Toggle spinners by adding/removing a class instead of show()/hide()

WordPress Trac noreply at wordpress.org
Mon Sep 23 20:30:29 UTC 2013


#22839: Toggle spinners by adding/removing a class instead of show()/hide()
--------------------------------------------+-----------------------------
 Reporter:  cdog                            |       Owner:
     Type:  defect (bug)                    |      Status:  new
 Priority:  normal                          |   Milestone:  Future Release
Component:  Administration                  |     Version:  3.4.2
 Severity:  normal                          |  Resolution:
 Keywords:  ui-focus 3.7-early needs-patch  |
--------------------------------------------+-----------------------------
Changes (by helen):

 * keywords:  has-patch needs-testing ui-focus 3.7-early => ui-focus
     3.7-early needs-patch
 * milestone:  3.7 => Future Release


Comment:

 Punting - patch magically still applies but doesn't seem to have been
 tested. It breaks at least the following - I only tested about half of the
 spinner instances listed in [comment:9].

 * Widgets, both individual when saving a change and sidebar when moving
 * Date format custom entry on general settings
 * Nav menu item adding (this shows up a LOT in that file, considering that
 it only appears to display for that one action - might we have some cruft
 or janky JS?)

--
Ticket URL: <http://core.trac.wordpress.org/ticket/22839#comment:29>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list