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
[reorg] New sidebar! #8239
base: main
Are you sure you want to change the base?
[reorg] New sidebar! #8239
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
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.
Hello, I am back! This is truly awesome. Thank you for spearheading this.
I added some comments about wording and suggested the addition of a sentence. I also made some edits for consistency's sake for sidebar title formatting!
Hope it helps!
{ text: 'Error Reference', slug: 'reference/error-reference', key: 'reference/error-reference' }, | ||
|
||
|
||
{ text: 'Other Develoment APIs', header: true, type: 'api', key: 'dev' }, |
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.
{ text: 'Other Develoment APIs', header: true, type: 'api', key: 'dev' }, | |
{ text: 'Other Astro APIs', header: true, type: 'api', key: 'dev' }, |
Note: Typo here if we keep it as "Development".
I think it would help to make it obvious that this is a section for developer tools.
Perhaps "Astro" APIs, is not correct, though. Maybe "Extend Astro" or "Create an Integration" could also work.
|
||
If you aren't able to open your project in the browser, go back to the terminal where you ran the `dev` command and look to see if an error occurred, or if your project is being served at a different URL than the one linked to above. | ||
If all goes well, you should see a success message followed by some recommended next steps. `cd` into your new project directory to begin using Astro. |
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.
If all goes well, you should see a success message followed by some recommended next steps. `cd` into your new project directory to begin using Astro. | |
If all goes well, you will see a success message followed by some recommended next steps. `cd` into your new project directory to begin using Astro. |
Perhaps a more hopeful option!
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 really love the way you have laid this out. It is quite encompassing yet concise!
</TabItem> | ||
</Tabs> | ||
|
||
Astro will build a deploy-ready version of your site in a separate folder (`dist/` by default) and you can watch its progress in the terminal. This will alert you to any build errors in your project before you deploy to production. |
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.
Astro will build a deploy-ready version of your site in a separate folder (`dist/` by default) and you can watch its progress in the terminal. This will alert you to any build errors in your project before you deploy to production. | |
Astro will build a deploy-ready version of your site in a separate folder (`dist/` by default) and you can watch its progress in the terminal. This will alert you to any build errors in your project before you deploy to production. If you opted to use Typescript, the `build` script will also check your project for type errors. |
Not sure of the accuracy here, but perhaps important to mention. I remember this being discussed quite a bit ago, and how it is not abundantly clear that the create
command adds this to your package.json
when using Typescript.
@@ -37,11 +37,9 @@ Check out the following courses and tutorials to learn more about Astro. | |||
|
|||
## Recipes and Guides | |||
|
|||
See guided examples of adding features to your Astro project, submitted by our community members! | |||
[Astro's official recipes](/en/recipes/) are short, focused how-to guides that walk a reader through completing a working example of a specific task. Recipes are a great way to add new features or behavior to your Astro project by following step-by-step instructions! |
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.
[Astro's official recipes](/en/recipes/) are short, focused how-to guides that walk a reader through completing a working example of a specific task. Recipes are a great way to add new features or behavior to your Astro project by following step-by-step instructions! | |
[Astro's official recipes](/en/recipes/) are quick, focused how-to guides that walk a reader through completing a working example of a specific task. Recipes are a great way to add new features or behavior to your Astro project by following step-by-step instructions! |
I think this would make it sound like the recipes are concise, rather than simple, and perhaps encourage more people to use them. I might ALSO be overthinking!
|
||
{ text: 'Guides', header: true, type: 'learn', key: 'features' }, | ||
|
||
{ text: 'Create Routes', header: true, type: 'learn', key: 'routes' }, |
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.
{ text: 'Create Routes', header: true, type: 'learn', key: 'routes' }, | |
{ text: 'Create routes', header: true, type: 'learn', key: 'routes' }, |
slug: 'basics/astro-components', | ||
key: 'basics/astro-components', | ||
}, | ||
{ text: 'Learn the Basics', header: true, type: 'learn', key: 'basics' }, |
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.
{ text: 'Learn the Basics', header: true, type: 'learn', key: 'basics' }, | |
{ text: 'Learn the basics', header: true, type: 'learn', key: 'basics' }, |
{ text: 'Installation and Setup', slug: 'install/auto', key: 'install/auto' }, | ||
{ text: 'Deploy your site', slug: 'guides/deploy', key: 'guides/deploy' }, | ||
{ text: 'Migrate to Astro', slug: 'guides/migrate-to-astro', key: 'guides/migrate-to-astro' }, | ||
{ text: 'Introductory Tutorial - Build a Blog', slug: 'tutorial/0-introduction', key: 'blog-tutorial' }, | ||
|
||
{ text: 'Core Concepts', header: true, type: 'learn', key: 'coreConcepts' }, |
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.
{ text: 'Core Concepts', header: true, type: 'learn', key: 'coreConcepts' }, | |
{ text: 'Core concepts', header: true, type: 'learn', key: 'coreConcepts' }, |
{ text: 'Connect your data', header: true, type: 'learn', key: 'data' }, | ||
{ text: 'Data Fetching', slug: 'guides/data-fetching', key: 'guides/data-fetching' }, | ||
{ text: 'Astro DB', slug: 'guides/astro-db', key: 'guides/astro-db' }, | ||
{ text: 'Add backend services', slug: 'guides/backend', key: 'guides/backend' }, |
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.
{ text: 'Add backend services', slug: 'guides/backend', key: 'guides/backend' }, | |
{ text: 'Add Backend Services', slug: 'guides/backend', key: 'guides/backend' }, |
}, | ||
{ text: 'Content Collections', slug: 'guides/content-collections', key: 'guides/content-collections' }, | ||
{ text: 'Connect a CMS', slug: 'guides/cms', key: 'guides/cms' }, | ||
{ text: 'Add an RSS feed', slug: 'guides/rss', key: 'guides/rss' }, |
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.
{ text: 'Add an RSS feed', slug: 'guides/rss', key: 'guides/rss' }, | |
{ text: 'Add an RSS Feed', slug: 'guides/rss', key: 'guides/rss' }, |
possibly slightly re-order the upper-level sections to be like this:
I bumped up “Manage your Content” and “Assets” because I personally need to reference them more. Creating routes I only need IF I’m doing dynamic routing. I also noticed that image and asset topics seem to pop up more in support than routing it feels like. |
Description (required)
This PR attempts to tame our wild sidebar!
This is a WIP and feedback is welcome! The goals of this PR:
Non-goals of this PR right now (although they could be future goals!)
Notable strategies I have taken with this PR:
Changing from headings emphasizing content type to headings describing topics included.
Rethinking the "new user journey".
Eliminating some things from the sidebar nav entirely
FINAL THOUGHTS
This is very much a work in progress, and feedback/thoughts are WELCOME! I may have not yet thought of things, or some things may yet be only in my head! This is only a starting point!