StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "fix bug in chrome with anchors not scrolling into view on new pages. added a 100ms delay to the scroll offset because chrome does not immediately scroll the page to the anchor on page load." into klp-docs

Browse Source
This commit is contained in:
Scott Main
2014-06-03 02:06:33 +00:00
committed by Android (Google) Code Review
parent 518ed145b1 06f3f2c148
commit 7a7abbdb10
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tools/droiddoc/templates-sdk/assets/js/docs.js
View File

@@ -494,7 +494,11 @@ false; // navigate across topic boundaries only in design docs
} }
} }
// Resize once loading is finished
resizeNav(); resizeNav();
// Check if there's an anchor that we need to scroll into view.
// A delay is needed, because some browsers do not immediately scroll down to the anchor
window.setTimeout(offsetScrollForSticky, 100);
/* init the language selector based on user cookie for lang */ /* init the language selector based on user cookie for lang */
loadLangPref(); loadLangPref();
@@ -998,10 +1002,6 @@ $(window).scroll(function(event) {
$cardInfo.css({position: 'absolute', bottom:'0px', left:'0px', right:'0px', overflow:'visible'}); $cardInfo.css({position: 'absolute', bottom:'0px', left:'0px', right:'0px', overflow:'visible'});
}); });
// Resize once loading is finished
resizeNav();
// Check if there's an anchor that we need to scroll into view
offsetScrollForSticky();
}); });
})(); })();
@@ -2525,7 +2525,6 @@ function offsetScrollForSticky() {
// to be because we need to move it down 60px to become in view), then move it down 60px // to be because we need to move it down 60px to become in view), then move it down 60px
if (Math.abs($matchingElement.offset().top - $(window).scrollTop()) < 20) { if (Math.abs($matchingElement.offset().top - $(window).scrollTop()) < 20) {
$(window).scrollTop($(window).scrollTop() - 60); $(window).scrollTop($(window).scrollTop() - 60);
} else {
} }
} }
} }