-
Notifications
You must be signed in to change notification settings - Fork 3.1k
docs: Restructure docs to a 8-Section Navigation with Guides #6335
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
gtag: { | ||
trackingID: "G-M3JWW8N2XQ", | ||
anonymizeIP: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this got added due to an error I was seeing in dev. Let me my find out if the anonymizeIP is needed.
docs/CLA.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved to the root since I did not see it rendered in the production. Not sure if it is in meant to be on the page or just a reference for contributors.
docs/netlify.toml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think all redirects should use redirects in the docusaurus config, any reason for extra ones here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we shouldn't put context providers under "Custom providers", since we have a separate idea of a "custom context provder" in the code for config.ts. These are more like "Built In" context providers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure I am following the suggestion, but the thought here is Context Provider are an advanced feature (for now) and we need to focus a story on core features while focus navigate the docs.
If you provide more specifics, I'd be happy to move them to wherever for now, but for context. Our docs are simply not being leveraged by users and Context providers is the 17/20th most visited page at 3k uniques in the last 10days. That is not popular by a far margin.
FWIW - Context in general has a bit of docs tech debt.
docs/docs/guides/best-practices.mdx
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flagging lorem ipsum stuff
docs/docs/guides/ollama-guide.mdx
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Flagging lorem ipsum
@@ -1,6 +1,6 @@ | |||
--- | |||
title: Creating an Organization | |||
description: Creating an Organization | |||
title: Creating An Organization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lower case is correct here for "an" in title
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
over zealous model here. I will need to fix a few of these.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can do this in the header of the md file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried this, and there are extra things the _category.json gives. It give every section a generated index overviews. These are needed to not have an undefined LLMs.txt
{
"label": "Guides",
"position": 6,
"link": {
"type": "generated-index",
"title": "Guides",
"description": "Learn how to use Continue with comprehensive guides and tutorials"
}
}
It also Title cases all the files without frontmatter.
With _category.json:
Without
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just confirmed the _category.json is need for a more expansive llms.txt. https://github.com/din0s/docusaurus-plugin-llms-txt?tab=readme-ov-file#documentation-structure
https://deploy-preview-6335--continuedev.netlify.app/llms.txt
docs/docs/advanced/00-overview.mdx
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome
# Core Features | ||
|
||
Continue provides four powerful AI-assisted development features that work seamlessly within your IDE. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels quite similar to /getting-started/overview
. Wondering if we should combine. If we did that, we could also then move /getting-started/install
out of a dropdown and into a top-level section in the sidebar which might be nice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This gets into the weeds in updating the actual content. I am going to track this in the "Quick Start" issue instead.
|
||
## Session Management | ||
|
||
- Use `Cmd/Ctrl + L` (VS Code) or `Cmd/Ctrl + J` (JetBrains) to start a new session |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use <kbd />
elements on other pages, e.g. <kbd>cmd</kbd> + <kbd>shift</kbd> + <kbd>P</kbd>
on https://docs.continue.dev/troubleshooting#vs-code
It renders a fancier looking keyboard key. I don't necessarily love them, but if we decide not to use them we should probably replace all instances of <kbd />
with regualr codeblock formatting like we use here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a linear issue to follow up on this. This was copied from an existing doc, and it will be easier to audit all of these instances in a single PR, but can update this instance since its a net new.
https://linear.app/continue/issue/CON-2176/keyboard-commands-in-docs-need-consistency-macos-windows
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right. It is not the same command/ctrl change to windows for VS Code specifically, which is why I started the linear issue. There is some nuance, which I will need to rethink how this command is shared, probably a TAB, but out of scope for this PR. I will track via Linear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I lean towards keeping this as a top level sidebar item, but that might be a maintainer specific viewpoint so don't feel strongly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Valid take. From the data (Posthog), it is not a top 20 clicked page. My guess is either folks don't make it to TroubleShooting, or search is enough. The hope is the LLMs.txt will be the solution eventually thanks to GEO.
Here's the list of the top 20 most visited docs pages on your site:
Homepage - 28,642 visits
Getting Started: Install - 10,069 visits
Customize: Deep Dives Autocomplete - 9,006 visits
Autocomplete: Model Setup - 8,745 visits
Chat: Model Setup - 8,139 visits
Getting Started: Overview - 8,095 visits
Agent: How to Use It - 7,206 visits
Reference - 7,146 visits
Chat: How to Use It - 5,398 visits
Customize: Model Providers - 5,061 visits
Agent: Model Setup - 4,256 visits
Autocomplete: How to Use It - 4,171 visits
Customize: Deep Dives MCP - 3,961 visits
Customize: Model Roles Reranking - 3,405 visits
Customize: Model Roles Chat - 3,289 visits
Customize: Overview - 3,211 visits
Customize: Context Providers - 3,129 visits
Customize: Model Providers Ollama - 2,972 visits
Chat: Context Selection - 2,924 visits
Customize: Deep Dives Codebase - 2,813 visits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to add the following to run this locally:
future: {
experimental_faster: true,
+ v4: true,
},
Probably just a local thing for me but curious if you had any issues with the experimental_faster
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge improvement! Really nice to be back down to a single sidebar.
Only high level comment I have is that I'm wondering if we should just fully remove the Chinese docs at this point. It doesn't seem feasible to keep them up to date with our work.
Also merged main to resolve a conflict from a PR I recently merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will address more of your comments and reviews later tonight @RomneyDa and @Patrick-Erichsen - thanks for taking a look
docs/netlify.toml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved
@@ -1,6 +1,6 @@ | |||
--- | |||
title: Creating an Organization | |||
description: Creating an Organization | |||
title: Creating An Organization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
over zealous model here. I will need to fix a few of these.
|
||
## Session Management | ||
|
||
- Use `Cmd/Ctrl + L` (VS Code) or `Cmd/Ctrl + J` (JetBrains) to start a new session |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's right. It is not the same command/ctrl change to windows for VS Code specifically, which is why I started the linear issue. There is some nuance, which I will need to rethink how this command is shared, probably a TAB, but out of scope for this PR. I will track via Linear.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Valid take. From the data (Posthog), it is not a top 20 clicked page. My guess is either folks don't make it to TroubleShooting, or search is enough. The hope is the LLMs.txt will be the solution eventually thanks to GEO.
Here's the list of the top 20 most visited docs pages on your site:
Homepage - 28,642 visits
Getting Started: Install - 10,069 visits
Customize: Deep Dives Autocomplete - 9,006 visits
Autocomplete: Model Setup - 8,745 visits
Chat: Model Setup - 8,139 visits
Getting Started: Overview - 8,095 visits
Agent: How to Use It - 7,206 visits
Reference - 7,146 visits
Chat: How to Use It - 5,398 visits
Customize: Model Providers - 5,061 visits
Agent: Model Setup - 4,256 visits
Autocomplete: How to Use It - 4,171 visits
Customize: Deep Dives MCP - 3,961 visits
Customize: Model Roles Reranking - 3,405 visits
Customize: Model Roles Chat - 3,289 visits
Customize: Overview - 3,211 visits
Customize: Context Providers - 3,129 visits
Customize: Model Providers Ollama - 2,972 visits
Chat: Context Selection - 2,924 visits
Customize: Deep Dives Codebase - 2,813 visits
updated the side nav to have Troubleshooting and Reference making it 8 items. Based on this interaction in Discord. https://discord.com/channels/1108621136150929458/1389889311457022004/1390021752117006336 ![]() |
2259f64
to
083ef5a
Compare
Description
[ What changed? Feel free to be brief. ]
fixes #6265
Restructured Continue's documentation from 10+ sections (with deep and confusing nesting) to 6 main sections for improved discoverability
https://screen.studio/share/cYqpX02F
Key Changes
Docs Structure Changes Only
No content was modified - only file locations, navigation structure, and internal linking. All copy remains unchanged.
How to Review
Navigate through the new 6-section structure and assess:
Checklist
Screenshots
[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Restructured the documentation into 6 main sections for easier navigation and better discoverability, replacing deep nesting and fixing over 50 broken links and images.