Support full TOC from a project and use for Guides project.#627
Support full TOC from a project and use for Guides project.#627
Conversation
|
@amotl Highly appreciate any support to properly release this together with the guide PR. |
|
Let's run a preview release in order to supply it to the other patch for testing purposes? |
| <!-- Show Guide sections as links in other projects --> | ||
| <li class="navleft-item"><a href="/docs/guide/home/">Overview</a></li> | ||
| <li class="navleft-item"><a href="/docs/guide/start/">Getting Started</a></li> | ||
| <li class="navleft-item"><a href="/docs/guide/guides/">Guides</a></li> |
There was a problem hiding this comment.
I am not particular happy with "naming things" about "Guides". I am considering it a positively tainted name per Diátaxis now, so let's find a different one.
There was a problem hiding this comment.
Maybe within that patch, the relocation of docs/index.md to docs/guides/index.md can be handled differently, or omitted at all? I know this solution is hacky as a whole, but maybe we can minimize its impact on this particular detail?
There was a problem hiding this comment.
Sure we can update the naming to support other new PRs already.
I'm not sure what the alternative would be for relocating the guides index? It seems a natural to me. We could even relocate the guides content to that folder if we want the physical structure to closer represent the TOC. But going forward I envision we will relocate many of the pages under Guides (or whatever it should be called)
There was a problem hiding this comment.
I'm not sure what the alternative would be for relocating the guides index? It seems a natural to me.
Understood, thanks.
Sure we can update the naming to support other new PRs already.
Thank you. Let's use handbook, unless you have any strong objections against, to get rid of the guide/guides redundancy and more?
<a href="/docs/guide/handbook/">Handbook</a>|
crate-docs-theme 0.41.0.dev0 has been released so the ingredients of this patch can be tested on downstream projects. It is not too difficult to publish a preview release in case you need it for subsequent iterations. sink:crate-docs-theme amo$ git diffdiff --git a/src/crate/theme/rtd/__init__.py b/src/crate/theme/rtd/__init__.py
index b045236..5fcfad4 100644
--- a/src/crate/theme/rtd/__init__.py
+++ b/src/crate/theme/rtd/__init__.py
@@ -23,9 +23,10 @@
import os
-VERSION = (0, 40, 1)
+VERSION = (0, 41, 0)
-__version__ = ".".join(str(v) for v in VERSION)
+#__version__ = ".".join(str(v) for v in VERSION)
+__version__ = "0.41.0.dev0"
__version_full__ = __version__make build
make upload |
|
We've built and uploaded |
amotl
left a comment
There was a problem hiding this comment.
Thanks again. Acknowledging by anticipating this will work out well. Please squash your commits and possibly revert the version bump.
| <!-- Show Guide sections as links in other projects --> | ||
| <li class="navleft-item"><a href="/docs/guide/home/">Overview</a></li> | ||
| <li class="navleft-item"><a href="/docs/guide/start/">Getting Started</a></li> | ||
| <li class="navleft-item"><a href="/docs/guide/handbook/">Handbook</a></li> |
There was a problem hiding this comment.
Asking again about my previous request: Maybe this can be adjusted to just point to guide/ again. Does anything speak against it?
<a href="/docs/guide/">Handbook</a>The relocation caused a problem:
There was a problem hiding this comment.
You are probably right in your assessment that this won't work as requested. Maybe try again? Otherwise we may find a different solution for mitigating the blank page?
There was a problem hiding this comment.
Would that not require that the handbook/index.md file is placed in the root where we currently have the index file that contains the 3 toplevel toc entries? Or am I missing something?
There was a problem hiding this comment.
That's probably the dilemma? Or just re-add the main body content there?
There was a problem hiding this comment.
Yeah - I was considering the same anyway as it's more clean. It just required a bit of special handling to ensure that the Overview page is selected/bolded initially. Fixed now.
…crate-docs-theme into bm/move-getting-started-1
|
Merging with the build error I expect fixed with a rebuild of the Guide with this PR. |
Add support for having the entire TOC of a project listed in the root, instead of under a single heading.
This requires coordinated release as 0.41.0 together with crate/cratedb-guide#348.
Preview
See preview in PR-348.