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

General documentation fixes #2463

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

onkoe
Copy link

@onkoe onkoe commented Mar 19, 2024

Discussion issue: #2426

This PR aims to fix some tonal and guidance inconsistencies within the Zola docs. For now, I won't make any sweeping changes, as tera2 has progressed quite a lot!

I'd be happy to focus on particular pages if anyone has any suggestions! :)

Copy link
Collaborator

@Keats Keats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry for the late review :(
It would be nice to revert the list change so it's easier to review

Zola uses the directory structure to determine the site structure.
Each child directory in the `content` directory represents a [section](@/documentation/content/section.md)
that contains [pages](@/documentation/content/page.md) (your `.md` files).
Zola determines your site's structure from its directories. Each child folder in the `content` directory represents a separate [section](@/documentation/content/section.md)[^1]that contains [pages](@/documentation/content/page.md) (your `.md` files).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

* `id`: the video id (mandatory)
* `playlist`: the playlist id (optional)
* `class`: a class to add to the `<div>` surrounding the iframe
* `autoplay`: when set to "true", the video autoplays on load
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we revert the - -> * changes? It's the same meaning and it causes unnecessary diff

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.

None yet

2 participants