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 site uses an old version of jquery #477

Closed
martinohanlon opened this issue Jul 11, 2023 · 8 comments
Closed

Docs site uses an old version of jquery #477

martinohanlon opened this issue Jul 11, 2023 · 8 comments
Labels

Comments

@martinohanlon
Copy link
Collaborator

martinohanlon commented Jul 11, 2023

This PR (#468) was recently progressed to upgrade the version of jquery from 1.12.4 to 3.6.4.

At the time the change was pushed to master but the docs werent rebuilt. I have just tried to rebuild the docs and the change to jquery stops features like navigation working.

The jquery library is part of the cinder mkdocs theme (https://github.com/chrissimpkins/cinder) hasnt seen an update for 3 years and no recent activity.

Options seem to be:

  • revert the change and go back to an old version of jquery and the potential security issue
  • change the theme to a different and supported theme
  • upgrade jquery and try and fix the cinder issues (and then push upstream to cinder)

I would suggest reverting the change right now (as we can build docs at the moment) and looking for another theme.

@lawsie - thoughts?

@martinohanlon
Copy link
Collaborator Author

martinohanlon commented Jul 11, 2023

Raised issue with the cinder project - chrissimpkins/cinder#110

@martinohanlon
Copy link
Collaborator Author

If we decided to move to a new theme, the bootswatch project has a lot of good options.

Personally I like litera

@lawsie
Copy link
Owner

lawsie commented Jul 14, 2023 via email

@martinohanlon
Copy link
Collaborator Author

martinohanlon commented Jul 14, 2023

we probably want to move to
something that is maintained actively to avoid having to move yet again!

Yep, the bootstrap and bootswatch are the 2 noted on mkdocs and look to be well supported.

Suggest we back out the jQuery update so we can push changes and create a new branch where we can assess new mkdocs themes.

@martinohanlon
Copy link
Collaborator Author

I have backed out the jquery and done a release of the documentation

@martinohanlon martinohanlon changed the title Upgrade to jquery breaks docs site Docs site uses an old version of jquery Jul 18, 2023
@lawsie
Copy link
Owner

lawsie commented Jul 25, 2023

Just had a look and I prefer Quartz. (Just kidding, litera looks perfect...) - I'll have a play.

@martinohanlon
Copy link
Collaborator Author

Resolved with PR #481

@martinohanlon
Copy link
Collaborator Author

Resolved.

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

No branches or pull requests

2 participants