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

[Bug] Modern template does not support scroll spy #8539

Open
chuongmep opened this issue Mar 20, 2023 · 5 comments
Open

[Bug] Modern template does not support scroll spy #8539

chuongmep opened this issue Mar 20, 2023 · 5 comments
Labels
bug A bug to fix template The stock site template

Comments

@chuongmep
Copy link
Contributor

Describe the bug

When we scroll mouse, content not follow by step.

image

To Reproduce
Steps to reproduce the behavior:

  1. Use new Modern templete
  2. See error

Expected behavior
It will be look good like default templete

image

Screenshots
If applicable, add screenshots to help explain your problem.

Context (please complete the following information):

  • Browser: [e.g. chrome, safari, edge]
  • Form factor: [e.g. desktop, mobile]
  • Docfx version: 2.63.0
  • Template: [e.g. default, statictoc]

Additional context
Add any other context about the problem here.

@chuongmep chuongmep added bug A bug to fix template The stock site template labels Mar 20, 2023
@yufeih yufeih changed the title [Bug] Template bug Modern have small bug with content [Bug] Modern template does not support scroll spy Mar 21, 2023
@yufeih
Copy link
Contributor

yufeih commented Mar 21, 2023

Starting from bootstrap 5.2, the scrollspy becomes pretty unusable: twbs/bootstrap#36431
We can wait for a fix to land in bootstrap or use a different scrollspy library.

@MJECloud
Copy link

Any updates on this? 😊

@chuongmep
Copy link
Contributor Author

Hi @MJECloud , this is problem resolved

@MJECloud
Copy link

Ah okay, so it's resolved by rendering all headlines?
For example on the docfx website it doesn't highlight where you are.
Which works in the Default Template.

image

@chuongmep
Copy link
Contributor Author

I think we should open another issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug to fix template The stock site template
Projects
None yet
Development

No branches or pull requests

3 participants