Goyslop/4chan-x
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix short threads at the bottom of the page not being selectable by keybinds. #265

Browse Source
This commit is contained in:
ccd0 2014-12-31 00:12:13 -08:00
parent ac975bcbbc
commit ae8e1c2225
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/Miscellaneous/Nav.coffee
View File

@ -58,4 +58,19 @@ Nav =
# or we're above the first thread and don't want to skip it. # or we're above the first thread and don't want to skip it.
top = Header.getTopOf thread top = Header.getTopOf thread
thread = next if delta is +1 and top < 5 or delta is -1 and top > -5 thread = next if delta is +1 and top < 5 or delta is -1 and top > -5
# Add extra space to the end of the page if necessary so that all threads can be selected by keybinds.
extra = Header.getTopOf(thread) + doc.clientHeight - d.body.getBoundingClientRect().bottom
d.body.style.marginBottom = "#{extra}px" if extra > 0
Header.scrollTo thread Header.scrollTo thread
if extra > 0 and !Nav.haveExtra
Nav.haveExtra = true
$.on d, 'scroll', Nav.removeExtra
removeExtra: ->
extra = doc.clientHeight - d.body.getBoundingClientRect().bottom
if extra > 0
d.body.style.marginBottom = "#{extra}px"
else
d.body.style.marginBottom = null
delete Nav.haveExtra
$.off d, 'scroll', Nav.removeExtra