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

Use different toctrees for different sections #6240

Merged
merged 1 commit into from
Dec 10, 2023

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Dec 10, 2023

It will make clearer what category every article falls into. Also it addresses an issue that results in the second section disappearing whenever you click on any link.
It reveals classification issues in the existing docs. For instance, I think architecture should go in the "Explanation" section, and that there should probably be a tutorial and how-to section. If we decide to do such moves, we should probably set up redirects, and I do not know how to achieve that.

I reused syntax I saw in the ORM docs, I think it should do the trick.

Here is a gif showing the bug:

Peek 2023-12-10 18-47

It will make clearer what category every article falls into. Also it
addresses an issue that results in the second section disappearing
whenever you click on any link.
It reveals classification issues in the existing docs. For instance, I
think architecture should go in the "Explanation" section, and that
there should probably be a tutorial and how-to section. If we decide to
do such moves, we should probably set up redirects, and I do not know
how to achieve that.
@greg0ire
Copy link
Member Author

greg0ire commented Dec 10, 2023

Nice, recent article by @weierophinney that also hints at moving "Architecture": https://mwop.net/blog/2023-12-10-advent-dbal.html

@greg0ire greg0ire merged commit 549f54d into doctrine:3.7.x Dec 10, 2023
@greg0ire greg0ire deleted the doc-headers branch December 10, 2023 18:25
@derrabus derrabus added this to the 3.7.3 milestone Feb 3, 2024
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