Skip to content

Conversation

@AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Oct 20, 2024

Alternative to #125754, for discussion.

A


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

@AA-Turner AA-Turner added skip issue skip news docs Documentation in the Doc dir labels Oct 20, 2024
@willingc
Copy link
Contributor

@AA-Turner I love it! Thank you for your Sphinx expertise. I'm going to close #125754 since this exceeds my expectations. 🎉

@AA-Turner AA-Turner marked this pull request as ready for review October 20, 2024 22:59
@willingc
Copy link
Contributor

I've been going through this page by page and it's a definite enhancement on most pages. I would recommend checking out the os module and built-in types to see if offering the ability to collapse a level would make sense, or if there is another way, short of a righthand sidebar that would shorten those pages. It may be that some of those pages could be split.

@AA-Turner
Copy link
Member Author

I'll merge this for now as it's had some support here and on Discourse, but if it turns out to be too disruptive we can revert quite easily.

A

@AA-Turner AA-Turner added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Oct 22, 2024
@AA-Turner AA-Turner merged commit 91ddde4 into python:main Oct 22, 2024
29 checks passed
@miss-islington-app
Copy link

Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

@AA-Turner AA-Turner deleted the docs/objdesc-toc branch October 22, 2024 13:07
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2024
@bedevere-app
Copy link

bedevere-app bot commented Oct 22, 2024

GH-125840 is a backport of this pull request to the 3.13 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 22, 2024
@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Oct 22, 2024
@bedevere-app
Copy link

bedevere-app bot commented Oct 22, 2024

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

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Oct 22, 2024
AA-Turner added a commit that referenced this pull request Oct 22, 2024
AA-Turner added a commit that referenced this pull request Oct 22, 2024
hugovk added a commit to hugovk/cpython that referenced this pull request Jan 2, 2025
@hugovk
Copy link
Member

hugovk commented Jan 2, 2025

I'll merge this for now as it's had some support here and on Discourse, but if it turns out to be too disruptive we can revert quite easily.

A

Unfortunately this has proved a bit too disruptive, I've opened #128406 to revert.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 2, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 2, 2025
AA-Turner pushed a commit that referenced this pull request Jan 2, 2025
AA-Turner pushed a commit that referenced this pull request Jan 2, 2025
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
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