Thanks for pointing this issue out. Strange, I remember it used to be working. Do you have any idea why it stopped working?

Thanks for pointing this issue out.

No problem.

Strange, I remember it used to be working.

Personally, I don't particularly remember the website ever having the feature,
and I've known about EditorConfig for at least a few years. Though the visual
effect is usually rather subtle (with good reason), so it might just be that I
never paid much attention to it.

Do you have any idea why it stopped working?

Not really. Before opening the PR, I noticed that Scrollspy still works on the
Bootstrap docs even on the 1.4.0 version, so out of curiosity I had tried
replacing js files on the website with the same exact same ones used on the
docs:

• https://getbootstrap.com/1.4.0/
• https://code.jquery.com/jquery-1.5.2.min.js
• https://getbootstrap.com/1.4.0/assets/js/bootstrap-scrollspy.js

But it didn't work. I also tried the files suggested on the 4.6.0 docs (also
using the new markup attributes):

• https://getbootstrap.com/docs/4.6/getting-started/download/#jsdelivr
• https://code.jquery.com/jquery-3.5.1.slim.min.js
• https://cdn.jsdelivr.net/npm/bootstrap@4.6.0/dist/js/bootstrap.min.js

But even copying and pasting the entire first example from the docs (and making
the paragraphs long enough) didn't work:

• https://getbootstrap.com/docs/4.6/components/scrollspy/#example-in-navbar

There were no errors in the console in any case. Admittedly, I didn't try very
hard to debug it and even though I do find the feature to be a nice touch (like
how it's used in the Bootstrap docs), I'm more inclined towards just dropping
the includes, especially since currently they have no effect anyway.

(Rebased to master to fix conflict with #145)

Let me check & review later today 🙏

(Rebased to master to fix conflict with #147)

Let me check & review later today pray

Hello, feel free to let me know if you need anything else for the review.

@kmk3 @xuhdev
Sorry for the delay, I just checked everything!
As it turns out, this change seems to be fine.

This commit changed the URL format of the link, so scrollspy stopped working from this point on.

    <ol data-scrollspy="scrollspy">
-      <li><a href="#overview">What is EditorConfig?</a></li>
+      <li><a href="{{ site.baseurl }}/#overview">What is EditorConfig?</a></li>
    </ol>

To test this, I executed the code below in Chrome's developer tools.
This code will fix the link and refresh scrollspy to work again.

document.querySelectorAll('[data-scrollspy] > li > a').forEach(anchorTag => {
  const currentHref = anchorTag.getAttribute('href')
  anchorTag.setAttribute('href', currentHref.replace(/\/(.*)/, '$1'))
  console.log(currentHref)
})
$('body').scrollSpy('refresh')

But this solution will cause a problem (@kmk3 made this fix and reverted at #147) so disable scrollspy for now and re-implement in the future will be the best choice.

@Pittan commented 5 hours ago:

@kmk3 @xuhdev
Sorry for the delay, I just checked everything!
As it turns out, this change seems to be fine.

This commit
changed the URL format of the link, so scrollspy stopped working from this
point on.

    <ol data-scrollspy="scrollspy">
-      <li><a href="#overview">What is EditorConfig?</a></li>
+      <li><a href="{{ site.baseurl }}/#overview">What is EditorConfig?</a></li>
    </ol>

To test this, I executed the code below in Chrome's developer tools.
This code will fix the link and refresh scrollspy to work again.

document.querySelectorAll('[data-scrollspy] > li > a').forEach(anchorTag => {
  const currentHref = anchorTag.getAttribute('href')
  anchorTag.setAttribute('href', currentHref.replace(/\/(.*)/, '$1'))
  console.log(currentHref)
})
$('body').scrollSpy('refresh')

But this solution will cause a problem (@kmk3 made this fix and reverted
at #147) so disable scrollspy for now and re-implement in the future will be
the best choice.

Great analysis; I didn't know that you were going to bisect it.

I find it interesting that it was broken such a long time ago; wasn't expecting
that. I did the math and apparently it has actually been broken (commit
67a887a) for longer than it had been working (commit 36ad237):

$ git show --pretty='%h %ai %s' -s 36ad237 67a887a
36ad237 2012-01-20 07:17:34 -0800 Add left menu bar nav with scrollspy support
67a887a 2016-07-13 02:23:41 +0200 Add a Jekyll-based blog (#57)

# working
$ git show --pretty='%ad' --date=unix -s 36ad237
1327072654
$ git show --pretty='%ad' --date=unix -s 67a887a
1468369421
$ expr 1468369421 - 1327072654
141296767
$ printf '141296767 / 60 / 60 / 24 / 365\n' | bc -l
4.48049108954845256215

# broken
$ expr "$(date +'%s')" - 1468369421
152731657
$ printf '152731657 / 60 / 60 / 24 / 365\n' | bc -l
4.84308907280568239472

So roughly it had worked for 4 years and ~6 months and has been broken for 4
years and ~9 months.

Awesome! Thanks for figuring this out, @kmk3 and @Pittan !