Skip to content

Upgrade docsy to latest version#916

Merged
saurabhraghuvanshii merged 9 commits intolayer5io:masterfrom
Sbragul26:upgrade-docsy
Mar 25, 2026
Merged

Upgrade docsy to latest version#916
saurabhraghuvanshii merged 9 commits intolayer5io:masterfrom
Sbragul26:upgrade-docsy

Conversation

@Sbragul26
Copy link
Contributor

Notes for Reviewers

This PR fixes #895

  • Upgraded Docsy from v0.12.0 → v0.14.3
  • Fixed UI issues caused by the upgrade (navbar, sidebars, main page)
  • Restored old navbar behavior by updating _nav.scss
  • Cleaned up conflicting styles and fixed CSS issues

Signed commits

  • Yes, I signed my commits.

Sbragul26 and others added 5 commits March 20, 2026 14:42
Signed-off-by: Sbragul26 <sbragul26@gmail.com>
Signed-off-by: Sbragul26 <sbragul26@gmail.com>
Signed-off-by: Sbragul26 <sbragul26@gmail.com>
Signed-off-by: Sbragul26 <sbragul26@gmail.com>
@netlify
Copy link

netlify bot commented Mar 20, 2026

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 9d66826
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/69c4413c8937f60008e62b5a
😎 Deploy Preview https://deploy-preview-916--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Sbragul26 why this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added this because after upgrading Docsy, it now uses td/_nav.scss for navbar styles.
Our old navbar styles (in _nav.scss) were not getting applied, so I imported them here to make sure they work again.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screen-Recording.mp4

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, instead of importing, we could directly move our _nav.scss into td/. That should also work — just checking if that approach is preferred? I didn’t change that for now.

Signed-off-by: Sbragul26 <sbragul26@gmail.com>
@Bhumikagarggg
Copy link
Contributor

Bhumikagarggg commented Mar 22, 2026

@Sbragul26 Thank you for your contribution! Let's discuss this during the website call tomorrow at 5:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

Sbragul26 and others added 2 commits March 24, 2026 22:13
@saurabhraghuvanshii saurabhraghuvanshii merged commit 4b4328c into layer5io:master Mar 25, 2026
7 checks passed
@saurabhraghuvanshii
Copy link
Member

Thanks @Sbragul26 lgtm!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade from docsy 0.12.0 to the latest release

3 participants