Skip to content

Add GitHub repository links #1128

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

Merged
merged 2 commits into from
Jul 3, 2025
Merged

Add GitHub repository links #1128

merged 2 commits into from
Jul 3, 2025

Conversation

lindsaym-fa
Copy link
Contributor

@lindsaym-fa lindsaym-fa commented Jul 2, 2025

Reorganizes our docs header, sidebar, and homepage to make way for links to this repository.


Moves search button to the sidebar:
Screenshot 2025-07-02 at 5 39 21 PM


Which makes room for GitHub and Star links in the header:
Screenshot 2025-07-02 at 5 44 12 PM


Removes references to backers from the beta callout (I'll be milking this "bigger and beta" joke for as long as I can):
Screenshot 2025-07-02 at 5 40 07 PM


Updates the links lower on the homepage to include GitHub and Discord with a separate section for socials:
Screenshot 2025-07-02 at 5 39 37 PM

Copy link

vercel bot commented Jul 2, 2025

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

Name Status Preview Updated (UTC)
webawesome ✅ Ready (Inspect) Visit Preview Jul 3, 2025 2:27pm

@lindsaym-fa
Copy link
Contributor Author

@claviska, the Discord link shows that the invite is coming from me. Would you prefer we update this with an invite link from you instead? 😄
Screenshot 2025-07-02 at 5 45 24 PM

@lindsaym-fa lindsaym-fa requested a review from claviska July 2, 2025 21:46
@claviska
Copy link
Member

claviska commented Jul 3, 2025

Would you prefer we update this with an invite link from you instead? 😄

It doesn't matter to me as long as you're comfortable with it.

@claviska
Copy link
Member

claviska commented Jul 3, 2025

Hmm, there's a lot of real estate being wasted at the top of the mobile sidebar:

CleanShot 2025-07-03 at 09 02 21@2x

I wanted to see what it looked like if the selects were on the same row. By setting flex-wrap: nowrap on the <div class="cluster">, it looks like this:

CleanShot 2025-07-03 at 09 00 19@2x

There's less wasted space, but I wonder what it would look like to have the search up there too. 🤔

CleanShot 2025-07-03 at 09 09 40@2x

Here's what I did in the dev console to get here:

  • hid the display-input part of both selects
  • applied width: fit-content to the listbox part of each select
  • remove light DOM margin that caused extra space
  • moved mobile search into the same cluster

It's more compact, but I'm not sure I love it since you can't see the selected theme's label now. And the search should probably stretch closer to the close button.

I think I like the second image (with search below the header), but maybe you have some better ideas on how to clean this up.

@lindsaym-fa
Copy link
Contributor Author

Agreed that your second screenshot is a good balance between space usage and usability! I pushed some changes to replicate that.

@lindsaym-fa lindsaym-fa merged commit 5ce3467 into next Jul 3, 2025
2 checks passed
@lindsaym-fa lindsaym-fa deleted the lm/add-repo-links branch July 10, 2025 17:53
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