8.3.5 makes all navigation links unusable

[Antonin Kral]

Contribution guidelines

• I've read the contribution guidelines and wholeheartedly agree

I've found a bug and checked that ...

• ... the problem doesn't occur with the mkdocs or readthedocs themes
• ... the problem persists when all overrides are removed, i.e. custom_dir, extra_javascript and extra_css
• ... the documentation does not mention anything about my problem
• ... there are no open or closed issues that are related to my problem

Description

Updating to 8.3.5 renders all navigation links (left side, toc, main, search) non-reactive. They stop reacting to clicks and user cannot navigate the site.

Expected behaviour

Links are clickable

Actual behaviour

you cannot browse the site

Steps to reproduce

Just build the site with 3.8.5, going back to 3.8.4 resolves the issue.

Package versions

mike==1.1.2
mkdocs-material==8.3.5
mkdocs-material-extensions==1.0.3
mkdocs-minify-plugin==0.5.0
mkdocs-redirects==1.0.4

Python 3.10.5 but tested on other versions as well.

Configuration

site_name: Test documentation portal
theme:
  name: material

System information

Tested various browsers, all behave the same

[Martin Donath]

Thanks for reporting. Possibly related to the changes added in #4012. It's not happening on our documentation site, so I suspect it's related to some specific configuration. Please provide a minimal reproducible example.

[Antonin Kral]

TBH I am slightly puzzled. I am not able to reproduce on my local machine (mike serve as well as mkdocs serve), but it manifests itself when deployed via gitlab and gitlab pages.

I am still getting the issue with rather minimal configuration

site_name: IP Fabric Documentation Portal
theme:
  name: material

extra:
  version:
    provider: mike

plugins:
  - mike:
      version_selector: true

I will try setup a separate repo, so we don't need to butcher our production documentation :) The "staging" repo is available at https://gitlab.com/ip-fabric/staging-docs while the deployed pages are accessible via https://ip-fabric.gitlab.io/staging-docs/main/

[Martin Donath]

Ok, I'm stupid 🥴. I accidentally left ev.preventDefault() in the logic of the version selector which I only included for testing that I just removed in 4f78e81. Navigation should now work as expected. I'm very sorry for the inconvenience.

[Antonin Kral]

Dah, I have not tested static build. That would showed the behavior on my local machine as well.

4f78e81

Yup, that did the trick. Just redeployed with tip of the master and everything works as expected. Patch release would be great, so others don't face the same issue.

Thank you very much for your great work. Really appreciated!

[Martin Donath]

Okay, perfect, many thanks for testing! I'll push out a new release today.

[Martin Donath]

Released as part of 8.3.6.