Skip to content

Conversation

@recrwplay
Copy link
Collaborator

@recrwplay recrwplay commented Dec 12, 2025

WIP for DOCOPS-31

lidiazuin and others added 22 commits December 12, 2025 15:50
Co-authored-by: Mark Woulfe <mark.woulfe@neotechnology.com>
Not something we would want to use generally, but maybe we can allow an
exception here.
When the nav is hidden by default at narrow page widths, you can view
the nav by clicking an icon.

When the nav is opened and visible you can click links in the nav to
move between pages.

However, the `onClick` events for the version selector do not work. I
traced this to another event that was taking priority, which was that
any click anywhere in the html triggered the nav to close. Anywhere
except links to other pages, which always fire first.

This PR modifies the showNav function so that the nav is hidden if you
click outside the nav, but not in the nav. This means that the version
selector can be used.

Note that depending on the screen resolution it is not actually possible
to click outside the nav, because the nav fills the display area.

---------

Co-authored-by: Stefano Ottolenghi <stefano.ottolenghi@neo4j.com>
@netlify
Copy link

netlify bot commented Dec 12, 2025

Deploy Preview for neo4j-docs-ui ready!

Name Link
🔨 Latest commit 18b7baa
🔍 Latest deploy log https://app.netlify.com/projects/neo4j-docs-ui/deploys/693c3c7081d02900082f38a8
😎 Deploy Preview https://deploy-preview-350--neo4j-docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@recrwplay recrwplay changed the title Docs restructure DOCOPS-31 Docs restructure Dec 12, 2025
@recrwplay recrwplay changed the title DOCOPS-31 Docs restructure Docs restructure Dec 12, 2025
@recrwplay recrwplay marked this pull request as draft December 17, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants