lawsie / guizero

A Python 3 library to allow learners to quickly and easily create GUIs.
https://lawsie.github.io/guizero
BSD 3-Clause "New" or "Revised" License
406 stars 81 forks source link

Docs site uses an old version of jquery #477

Closed martinohanlon closed 1 year ago

martinohanlon commented 1 year ago

This PR (https://github.com/lawsie/guizero/pull/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:

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

@lawsie - thoughts?

martinohanlon commented 1 year ago

Raised issue with the cinder project - https://github.com/chrissimpkins/cinder/issues/110

martinohanlon commented 1 year ago

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

Personally I like litera

lawsie commented 1 year ago

I don’t mind moving, as long as it maintains friendliness. When I get a moment I’ll have a look at those 😁 I guess we probably want to move to something that is maintained actively to avoid having to move yet again!

On Fri, 14 Jul 2023 at 08:35, martinohanlon @.***> wrote:

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

Personally I like litera https://bootswatch.com/litera/

— Reply to this email directly, view it on GitHub https://github.com/lawsie/guizero/issues/477#issuecomment-1635432297, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHRRUIMAHOG4TCEXZYAUALXQDZDVANCNFSM6AAAAAA2GHBYCQ . You are receiving this because you were mentioned.Message ID: @.***>

martinohanlon commented 1 year ago

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 commented 1 year ago

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

lawsie commented 1 year ago

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

martinohanlon commented 1 year ago

Resolved with PR #481

martinohanlon commented 1 year ago

Resolved.