Skip to content

Conversation

@agriyakhetarpal
Copy link
Member

@agriyakhetarpal agriyakhetarpal commented Feb 4, 2026

The secondary (shortcuts) sidebar wasn't showing up on individual SPEC pages. The JavaScript code we have for shortcuts in the theme looks for headings that are direct children of .content-container, but specs/single.html had an extra <div class="article-content"> wrapper around the content. I have removed this wrapper since it isn't defined anywhere in the theme anyway.

@netlify
Copy link

netlify bot commented Feb 4, 2026

Deploy Preview for scientific-python-org ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 01f51d9
🔍 Latest deploy log https://app.netlify.com/projects/scientific-python-org/deploys/6982979c31469400094c27e0
😎 Deploy Preview https://deploy-preview-843--scientific-python-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 95
Accessibility: 100
Best Practices: 100
SEO: 100
PWA: -
View the detailed breakdown and full score reports

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

@agriyakhetarpal agriyakhetarpal changed the title Fix missing secondary sidebar on SPECs pages Fix missing secondary sidebars on SPECs pages Feb 4, 2026
@stefanv
Copy link
Member

stefanv commented Feb 4, 2026

Thank you for tracking this down, @agriyakhetarpal, much appreciated!

@stefanv stefanv merged commit d05c1e1 into scientific-python:main Feb 4, 2026
5 checks passed
@agriyakhetarpal agriyakhetarpal deleted the fix/spec-shortcuts branch February 4, 2026 10:35
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.

2 participants