[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