Merge pull request #60 from Fedomn/develop
fix(toc): fix inaccurate headerLinkTop calculation
This commit is contained in:
@@ -175,13 +175,12 @@ jQuery(function($) {
|
|||||||
};
|
};
|
||||||
changeTocState();
|
changeTocState();
|
||||||
|
|
||||||
const HEADERFIX = 120;
|
|
||||||
const $toclink = $('.toc-link');
|
const $toclink = $('.toc-link');
|
||||||
const $headerlink = $('.headerlink');
|
const $headerDummyLink = $('.post-dummy-target');
|
||||||
const $tocLinkLis = $('.post-toc-content li');
|
const $tocLinkLis = $('.post-toc-content li');
|
||||||
const activeIndex = function () {
|
const activeIndex = function () {
|
||||||
const scrollTop = $(window).scrollTop();
|
const scrollTop = $(window).scrollTop();
|
||||||
const headerlinkTop = $.map($headerlink, function(link) {
|
const headerlinkTop = $.map($headerDummyLink, function(link) {
|
||||||
return $(link).offset().top;
|
return $(link).offset().top;
|
||||||
});
|
});
|
||||||
const searchActiveTocIndex = function(array, target) {
|
const searchActiveTocIndex = function(array, target) {
|
||||||
@@ -191,7 +190,7 @@ jQuery(function($) {
|
|||||||
return array.length - 1;
|
return array.length - 1;
|
||||||
};
|
};
|
||||||
|
|
||||||
const activeTocIndex = searchActiveTocIndex(headerlinkTop, scrollTop + HEADERFIX);
|
const activeTocIndex = searchActiveTocIndex(headerlinkTop, scrollTop);
|
||||||
|
|
||||||
$($toclink).removeClass('active');
|
$($toclink).removeClass('active');
|
||||||
$($tocLinkLis).removeClass('has-active');
|
$($tocLinkLis).removeClass('has-active');
|
||||||
|
|||||||
Reference in New Issue
Block a user