doc: fix issues related to page scrolling#6641
Uh oh!
There was an error while loading. Please reload this page.
Conversation
silverwind commented May 11, 2016
@nodejs/website @nodejs/documentation Is the JS "hack" too much? While I think it's pretty solid, I think that at this point I'm ok with removing it if someone has concerns about it. |
eljefedelrodeodeljefe commented May 11, 2016
Just general ones. Haven't made it to review yet. If you can avoid it it would be better.... |
eljefedelrodeodeljefe commented May 11, 2016
Since it is using vanilla and well supported JS, I'd be okay with having this now and hopefully removing this later. |
doc/template.html Outdated
There was a problem hiding this comment.
If you go this route, may I suggest to throttle the listener to avoid horrible perfs janks?
There was a problem hiding this comment.
The handler is pretty lightweight, I don't think it could cause any janks, except on maybe ancient hardware.
There was a problem hiding this comment.
Isn't it still good practice to throttle it regardless?
There was a problem hiding this comment.
Depends on the use case. With throttling, some events won't get cancelled which results in scroll ticks going through. That's not something I want in this case, so every event would have to be cancelled.
Hmm, I think I'll drop the JS after all. It was just one user complaining about it, while probably more people agree to let the browser handle scrolling. |
Moved the sidebar to a fixed position and moved the positioning of the main content to the page's body, which results in back/forward navigation through hash links and search highlight working again. Fixes: nodejs#6637 Based on: nodejs#5716
silverwind commented May 11, 2016
Okay, CSS only change now to fix two issues. Please review! |
lpinca commented May 11, 2016
I've tested this on Chrome and mobile Safari, all good 👍 . |
eljefedelrodeodeljefe commented May 11, 2016
LGTM then |
Moved the sidebar to a fixed position and moved the main column to the page's body, which results in back/forward navigation through hash links and search highlight working again. Fixes: #6637Fixes: #6751 Based on: #5716 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Robert Lindstaedt <[email protected]>
silverwind commented May 17, 2016
Landed in bef1ec0. |
5 participants
Checklist
Description of change
Moved the sidebar to a fixed position and moved the positioning of the main content to the page's body, which results in back/forward navigation through hash links and search highlight working again.
Added a script to prevent the main column from unintentional scrolling when the sidebar is scrolled outside its boundaries.
Fixes: #6637
Based on: #5716