Skip to content

Added an download link in the top nav bar - #406

Open
M1st3r3 wants to merge 1 commit into
mainfrom
yanni/doc-58-add-a-download-link-to-the-docscaidoio-top-navigation
Open

M1st3r3 wants to merge 1 commit into
mainfrom
yanni/doc-58-add-a-download-link-to-the-docscaidoio-top-navigation

Conversation

@M1st3r3

@M1st3r3 M1st3r3 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Added a Download link to the top navigation bar. It currently uses the same styling as the other navigation links. Let me know if you'd like a different color/style to make it stand out more.

image

Summary by CodeRabbit

  • New Features
    • Added a Download link to the site navigation, giving visitors a direct route to Caido’s download page. The link is available alongside the existing navigation options and takes visitors to the download page when selected.

@M1st3r3
M1st3r3 requested a review from Corb3nik as a code owner October 27, 2026 12:59
@netlify

netlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for caido-documentation ready!

Name Link
🔨 Latest commit 174dc03
🔍 Latest deploy log https://app.netlify.com/projects/caido-documentation/deploys/6ae0a0274cfc92ca01c69c56
😎 Deploy Preview https://deploy-preview-406--caido-documentation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Some coding-guideline sources were skipped; the reason is listed next to each one. Other guidelines were applied as usual. On a public repository, sources from your configuration are listed without their names.

Skipped guideline sources: 1
a configured source — couldn't be fetched

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 57c670c4-e27b-4364-9c0f-a0af5e3afc76
Commits

Reviewing files that changed from the base of the PR and between c899c34 and 174dc03.

Files selected for processing: 1
  • .vitepress/config.mts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

Walkthrough

The VitePress navigation now includes a Download link to Caido’s download page with documentation-source UTM parameters. The item uses ^$ as its active-match pattern.

Changes: Download navigation

Layer / File(s) Summary
Add the Download navigation item
.vitepress/config.mts
The navigation configuration adds a link to Caido’s download page with documentation-source UTM parameters and the ^$ active-match pattern.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Feature

Merge Risk: Minimal · up to 174dc

The Download link is low-risk and ready to merge.

Architecture risk: Low · up to 174dc

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .vitepress/config.mts: Adds a “Download” navigation item linking to https://www.caido.io/download?utm_source=docs&utm_medium=nav, with ^$ as its active-match pattern.
Pre-merge checks: Passed 4
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: adding a Download link to the top navigation bar. It is concise and related to the changeset, despite the grammatical error in "an download."
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@Corb3nik Corb3nik left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let me know if you'd like a different color/style to make it stand out more.

Yep! If you can make it the same color as the Contact Sales / Download Caido buttons on the marketing website, that'd be great.

Image
  • Same red color
  • Looks like a rounded button
  • The download icon on the left of the text

This branch has not been deployed

No deployments
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