[wp-trac] [WordPress Trac] #18758: Click empty admin bar to scroll to top

WordPress Trac wp-trac at lists.automattic.com
Mon Dec 19 20:38:48 UTC 2011


#18758: Click empty admin bar to scroll to top
---------------------------------+-----------------------
 Reporter:  TJNowell             |       Owner:  ericmann
     Type:  task (blessed)       |      Status:  assigned
 Priority:  normal               |   Milestone:  3.4
Component:  Toolbar              |     Version:  3.3
 Severity:  normal               |  Resolution:
 Keywords:  has-patch 3.4-early  |
---------------------------------+-----------------------

Comment (by ericmann):

 I realized that a couple of my conditional statements were tricky to
 follow: `if (leapY < stopY) leapY = stopY; timer++;` actually caused me a
 couple of issues just reading back my code ... so they're now broken up
 and fixed a bit.

 I've also removed the arbitrary `stopY` variable.  Since we're always
 scrolling to the top, there's no reason to set that separately.

 New patch tested against trunk.  Only adding the unminified version in
 case we need to make further changes.

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


More information about the wp-trac mailing list