Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs affix subitem not highlighted on master #14872

Closed
XhmikosR opened this issue Oct 25, 2014 · 6 comments · Fixed by #14873
Closed

Docs affix subitem not highlighted on master #14872

XhmikosR opened this issue Oct 25, 2014 · 6 comments · Fixed by #14873
Labels

Comments

@XhmikosR
Copy link
Member

I think I've seen this issue again and it was fixed, if anyone finds it let me know.

2014-10-25_08-39-12

@XhmikosR XhmikosR added the docs label Oct 25, 2014
@carasmo
Copy link

carasmo commented Oct 25, 2014

Works fine:
screen shot 2014-10-25 at 10 35 07 am

What browser and version?

@XhmikosR
Copy link
Member Author

Doesn't matter, it's not a browser issue. Happens with all the browsers on master docs, not on the website.

@juthilo
Copy link
Collaborator

juthilo commented Oct 25, 2014

X-Ref: #13659

@juthilo
Copy link
Collaborator

juthilo commented Oct 25, 2014

Doesn't seem to be broken all the time, i.e. sometimes it doesn't work for the first section, sometimes it doesn't work for any sections and sometimes it works fine for all sections.

My testing suggests it works if I scroll down before the page finishes loading.

@juthilo
Copy link
Collaborator

juthilo commented Oct 25, 2014

Scrolling down before the page is ready only makes the first section on that page work though.

@juthilo
Copy link
Collaborator

juthilo commented Oct 25, 2014

Ok, I tried reverting d15444d because not much else changed in scrollspy.js and it does indeed seem to be the culprit.

Is there any other way to fix #13071?

/cc @hnrch02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants