Skip to content

Conversation

@amirhhashemi
Copy link
Collaborator

@amirhhashemi amirhhashemi commented Feb 6, 2025

  • I have read the Contribution guide
  • This PR references an issue (except for typos, broken links, or other minor problems)

Description(required)

This PR improves the Navigation docs by:

  1. Adding information about other methods of navigation like useNavigate and redirect
  2. Fixing ambiguity with the example: The examples used App as the component name, which might not be obvious that it's used in a Route. They also were more complex than they needed to.
  3. Adding more examples and information in general. Like adding an example of relative paths, making it clear that native anchor tags triggers a client-side navigation (unlike other frameworks), etc.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify
Copy link

netlify bot commented Feb 6, 2025

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit 195025c
🔍 Latest deploy log https://app.netlify.com/sites/solid-docs/deploys/67cb8416f9698d00086dabaa
😎 Deploy Preview https://deploy-preview-1061--solid-docs.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.

@LadyBluenotes LadyBluenotes enabled auto-merge (squash) March 5, 2025 20:30
@LadyBluenotes LadyBluenotes disabled auto-merge March 5, 2025 20:30
@LadyBluenotes LadyBluenotes merged commit 2cdf837 into solidjs:main Mar 7, 2025
2 of 4 checks passed
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.

2 participants