Skip to content

Conversation

@AR21SM
Copy link
Member

@AR21SM AR21SM commented Aug 29, 2025

Description

This PR addresses multiple UI bugs and configuration issues affecting the DISCOVER Cookbook website functionality and user experience.

Key Fixes

Version Banner Configuration

Fixed the version banner display issue by properly configuring the release variable in the Sphinx configuration. Added release = version immediately after retrieving the version from environment variables to resolve the pydata-sphinx-theme version banner problem referenced in pydata/pydata-sphinx-theme#1629.

# DISCOVER/conf.py
version = os.environ.get("WEBSITE_VERSION", "dev")
release = version  # Critical for version banner functionality

Unifies switcher styles in Switchers.css, fixes mobile-nav layout, patches class concat issues, repairs broken links, , and adds inline comments for better readability.

@netlify
Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for stupendous-kringle-a86e81 ready!

Name Link
🔨 Latest commit 89fbd65
🔍 Latest deploy log https://app.netlify.com/projects/stupendous-kringle-a86e81/deploys/68b1c6450ddcae000863ffe4
😎 Deploy Preview https://deploy-preview-389--stupendous-kringle-a86e81.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.

@AR21SM AR21SM marked this pull request as draft August 29, 2025 14:29
@AR21SM AR21SM marked this pull request as ready for review August 29, 2025 15:29
@AR21SM AR21SM marked this pull request as draft August 29, 2025 15:30
@AR21SM AR21SM marked this pull request as ready for review August 30, 2025 23:40
@OriolAbril OriolAbril merged commit 0eecc84 into numfocus:main Aug 31, 2025
4 checks passed
@AR21SM AR21SM changed the title Fix/main branch bugs fix(ui): switcher styling bugs, mobile navigation, and unknown version banner issue Aug 31, 2025
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