Skip to content

Bring community back#6791

Closed
liruifengv wants to merge 3 commits intowithastro:mainfrom
liruifengv:bring-community-back
Closed

Bring community back#6791
liruifengv wants to merge 3 commits intowithastro:mainfrom
liruifengv:bring-community-back

Conversation

@liruifengv
Copy link
Copy Markdown
Member

@liruifengv liruifengv commented Feb 6, 2024

Description (required)

Bring community back.

I noticed that this part was removed when the new docs were migrated to starlight.

I don't know if this was done on purpose. I brought it back.

Related issues & labels (optional)

  • Closes #
  • Suggested label:

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Feb 7, 2024 9:18am

@sarah11918 sarah11918 added the site improvement Some thing that improves the website functionality - ask @delucis for help! label Feb 8, 2024
@delucis
Copy link
Copy Markdown
Member

delucis commented Feb 9, 2024

Thanks for the PR @liruifengv and for noticing these details!

This was actually (at least semi 😁) intentional for a couple of reasons.

  1. We want to move away completely from using the right sidebar for these links as they are not accessible at smaller viewports.

  2. Now that we’ve started work on https://community.astro.build/ it is now the focus for links related to the community (although we should probably think about linking to THAT from docs somewhere).

That said, we didn’t really follow up on completing that work yet, so this is a good opportunity to discuss it — thanks for creating that opportunity!

I’ve posted a new issue with some details of what we’d want: #6845
Would love to hear your thoughts there and will close this PR for now.

@delucis delucis closed this Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

site improvement Some thing that improves the website functionality - ask @delucis for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants