Skip to content

Upgrade docsy to latest version#916

Open
Sbragul26 wants to merge 5 commits intolayer5io:masterfrom
Sbragul26:upgrade-docsy
Open

Upgrade docsy to latest version#916
Sbragul26 wants to merge 5 commits intolayer5io:masterfrom
Sbragul26:upgrade-docsy

Conversation

@Sbragul26
Copy link

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 e83c0e8
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/69bd43bc2bb9b600086cf180
😎 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
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
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
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.

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

2 participants