Skip to content

Navbar: Learn links to /docs and docs pages no longer crash - #2347

Merged
sanjay-kv merged 1 commit into
mainfrom
redesign/navbar
Oct 20, 2026
Merged

sanjay-kv merged 1 commit into
mainfrom
redesign/navbar

Conversation

@sanjay-kv

@sanjay-kv sanjay-kv commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Docs pages threw Cannot read properties of null (reading 'clientHeight') because Docusaurus' table of contents highlight reads the height of an element with the class navbar, which the new navbar did not have. Added an invisible, correctly sized .navbar probe inside the new nav (CSS beats the legacy global .navbar rules with !important). This is why clicking "GitHub tutorials" and "recode hive docs" in the Learn menu looked broken.
  • The Learn label now links to /docs (like Showcase links to /showcase); the chevron next to it opens the panel. On mobile the Learn group gets a "Learn overview" link.

Checked

Build passes. In a browser: Learn label goes to /docs, "recode hive docs" goes to /docs, "GitHub tutorials" goes to /docs/GitHub/intro-github with no crash, and the nav stays 64px.

…ing .navbar element

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@sanjay-kv
sanjay-kv requested a review from Adez017 as a code owner October 20, 2026 15:29
@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
recode-website Ready Ready Preview Oct 20, 2026 3:30pm UTC

@github-actions

Copy link
Copy Markdown

Thank you for submitting your pull request! 🙌 We'll review it asap. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, if it's a Video -LEGEND, which helps speed up the review. If you have questions, reach out to LinkedIn. You can see in our Leaderboard in 20 min from now!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

  1. Get free Consultation after 5 PR: Mentorship for free.

  2. Get the Ebook for free after 10 PR: You can tag me in Discussion to get the same. Data Science cheatsheet for Beginners.

  3. Check out this weekly Newsletter: Sanjay's Newsletter.

Thanks again for your contribution! 😊

@github-actions github-actions Bot added in-review The current changes are in review and would need approval and testing before merging level 1 10 points recode this is label for leaderboard labels Oct 20, 2026
@github-actions github-actions Bot added this to the recode:launch 3.0 milestone Oct 20, 2026
@sanjay-kv
sanjay-kv merged commit 7c3fd4f into main Oct 20, 2026
9 checks passed
@github-actions

Copy link
Copy Markdown

🎉 Your pull request has been successfully merged! 🎉 Thank you for your valuable contribution to our project.

I would be grateful if you can nominate for keeping this opensource project live and running Link to Nominate. Please do star the repo and Keep up the fantastic work! 🚀

This branch was successfully deployed

1 active deployment
Preview — 26b2290f Deployed Oct 20, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in-review The current changes are in review and would need approval and testing before merging level 1 10 points recode this is label for leaderboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant