Skip to content

Fix: Move frontpage to top-level navigation for easier access #96

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

khushishukla2813
Copy link
Contributor

📝 Description

  • Moved the frontpage (Intro to DUB) to the top-level navigation.
  • This change eliminates the unnecessary nesting, making it easier to access the full documentation without multiple back clicks.
  • Fixed navigation behavior that caused confusion, especially on mobile devices.

📱 Mobile Improvement

  • Previously, navigation started at the second-level, making it cumbersome on mobile.
  • Now, the homepage is directly accessible, eliminating the need to click the back button multiple times.

🎨 Proposed New Design

I have also designed an improved version of the page with better navigation.
✅ If you like it, I would be happy to proceed with that design as a future enhancement.
image

📂 Files Changed:

  • mkdocs.yml

🔗 Reference Issue:

Closes #88

Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for dub-registry ready!

Name Link
🔨 Latest commit 8cfce9f
🔍 Latest deploy log https://app.netlify.com/sites/dub-registry/deploys/67dd9d82f1cae9000896a16a
😎 Deploy Preview https://deploy-preview-96--dub-registry.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 site configuration.

@@ -71,8 +71,8 @@ extra_css:
- stylesheets/extra.css

nav:
- "Intro to DUB": index.md # Moved to top level
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- "Intro to DUB": index.md # Moved to top level
- "Intro to DUB": index.md

putting in a comment will just confuse people reading the code in the future when they expect it to be top level and haven't seen it anywhere else

@WebFreak001
Copy link
Member

perhaps this side panel should be removed (for the front page only)

image

do you think you can find it in the docs in https://squidfunk.github.io/mkdocs-material/ ?

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.

Make dub.pm frontpage a top-level page
2 participants