jyapayne/docsify
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: scroll position issue, fixed #234

Browse source
This commit is contained in:
qingwei.li 2017-09-07 23:04:58 +08:00
commit 388ed3d685
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/core/event/scroll.js
View file

@ -6,6 +6,7 @@ const nav = {}
let hoverOver = false let hoverOver = false
let scroller = null let scroller = null
let enableScrollEvent = true let enableScrollEvent = true
let coverHeight = 0
function scrollTo (el) { function scrollTo (el) {
if (scroller) scroller.stop() if (scroller) scroller.stop()
@ -27,7 +28,7 @@ function highlight () {
const wrap = dom.find(sidebar, '.sidebar-nav') const wrap = dom.find(sidebar, '.sidebar-nav')
let active = dom.find(sidebar, 'li.active') let active = dom.find(sidebar, 'li.active')
const doc = document.documentElement const doc = document.documentElement
const top = doc && doc.scrollTop || document.body.scrollTop const top = (doc && doc.scrollTop || document.body.scrollTop) - coverHeight
let last let last
for (let i = 0, len = anchors.length; i < len; i += 1) { for (let i = 0, len = anchors.length; i < len; i += 1) {
@ -71,7 +72,9 @@ function highlight () {
} }
export function scrollActiveSidebar (router) { export function scrollActiveSidebar (router) {
if (isMobile) return const cover = dom.find('.cover.show')
coverHeight = cover ? cover.offsetHeight : 0
const sidebar = dom.getNode('.sidebar') const sidebar = dom.getNode('.sidebar')
const lis = dom.findAll(sidebar, 'li') const lis = dom.findAll(sidebar, 'li')
@ -89,6 +92,8 @@ export function scrollActiveSidebar (router) {
if (href) nav[decodeURIComponent(href)] = li if (href) nav[decodeURIComponent(href)] = li
} }
if (isMobile) return
dom.off('scroll', highlight) dom.off('scroll', highlight)
dom.on('scroll', highlight) dom.on('scroll', highlight)
dom.on(sidebar, 'mouseover', () => { hoverOver = true }) dom.on(sidebar, 'mouseover', () => { hoverOver = true })