Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: skip python-docs-theme 2023.7 to fix mobile menu #107666

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Aug 5, 2023

Skip python-docs-theme 2023.7 because python/python-docs-theme#141 means the menu doesn't work properly for mobile. Upstream report: python/python-docs-theme#144.

To test:

  1. Open docs in mobile view
  2. Click the menu at the top left

Expected result: menu opens (https://cpython-previews--107666.org.readthedocs.build/en/107666/)
Actual result: menu doesn't open (https://docs.python.org/3.13/)


📚 Documentation preview 📚: https://cpython-previews--107666.org.readthedocs.build/

@hugovk hugovk added skip issue skip news needs backport to 3.11 only security fixes needs backport to 3.12 bug and security fixes docs Documentation in the Doc dir labels Aug 5, 2023
@hugovk hugovk merged commit 9641c4d into python:main Aug 6, 2023
22 checks passed
@hugovk hugovk deleted the docs-mobile-menu branch August 6, 2023 11:08
@miss-islington
Copy link
Contributor

Thanks @hugovk for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-107690 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 bug and security fixes label Aug 6, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 6, 2023
(cherry picked from commit 9641c4d)

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
@bedevere-bot
Copy link

GH-107691 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Aug 6, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 6, 2023
(cherry picked from commit 9641c4d)

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
hugovk added a commit that referenced this pull request Aug 6, 2023
) (#107691)

Docs: skip python-docs-theme 2023.7 to fix mobile menu (GH-107666)
(cherry picked from commit 9641c4d)

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Yhg1s pushed a commit that referenced this pull request Aug 6, 2023
) (#107690)

Docs: skip python-docs-theme 2023.7 to fix mobile menu (GH-107666)
(cherry picked from commit 9641c4d)

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
hugovk added a commit to python/python-docs-theme that referenced this pull request Aug 14, 2023
hugovk added a commit to hugovk/python-docs-theme that referenced this pull request Aug 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants