-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update Output Formats documentation (v0.38) #404
Conversation
62dfcb7
to
43ab113
Compare
content/templates/output-formats.md
Outdated
By default, **all** pages will render to the `HTML` output format. Every `Page` | ||
also has a [`Kind`][page kinds]. Except for the `page` `Kind`, by default, all | ||
other *Kinds* will also render to the `RSS` output format. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the language here is a little technical.
I would suggest something in the line of: "By default, all ... HTML ... Also, every list page will, by default, render to the RSS
output format."
"list page" may not be perfect, but it is easy for people to understand.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But it's clearer that way. People will anyways needs to understand Kinds to be able to configure outputs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And also, doesn't feel right mis-documenting.. RSS, 404, etc kinda are not "list pages".. I think that's what you were inferring too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have attempted to make this a bit more clearer in my last force-pushed commit to this PR, while retaining the correctness.
content/templates/output-formats.md
Outdated
|
||
{{% note %}} | ||
If you add an `outputs` config section, the *output format* for all the | ||
unspecified *Kinds* will be set to only `HTML`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you take the "half-empty" approach to this. I would much prefer something in the line of:
If you add an
outputs
config section, you replace the Hugo defaultoutput
configuration with your own and must provide a complete configuration. You will getHTML
for any missing pageKind
entry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done (this, and also the next hunk where you commented).
content/templates/output-formats.md
Outdated
* The names used must match the `Name` of a defined `Output Format`. | ||
* Any `Kind` without a definition will default to `HTML`. | ||
* The output definition is per [`Page` `Kind`][page kinds] (i.e, `page`, `home`, `section`, `taxonomy`, or `taxonomyTerm`). | ||
* Any `Kind` not defined in this `outputs` config section will default to `HTML`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have already said this before.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, will remove it from here.
43ab113
to
5e60068
Compare
Even though the .Kind home, page and section are rather obvious. I have been and saw a lot of people being confused as to what is a home: the homepage I use the word entry to avoid common confusion surrounding the word |
466ff96
to
ec722c6
Compare
@regisphilibert Thanks for the feedback. I have now rephrased that part taking in your input. |
I think this wording is confusing. I'm sure not every body agrees on what a taxonomy is, but the documentation clearly states how a term relates to a taxonomy using its movie example. Following this logic: A
While the |
Also I believe the evocation of the layout and content file is a bit confusing as it does not always follow the naming logic of the page kinds. This is the simple approach I would recommend: home page section taxonomy taxonomyTerm |
ec722c6
to
fe001f9
Compare
How about now? |
Yes :) I still have reservation about the HTML structure illustrations as not everybody goes looking under the public/ hood. But that’s just my view. |
I didn't understand the reference. Update: Ah OK, you mean the Update 2: For ease of understanding.. may be remove the |
The reason why I don't think we should make any parallel with the url structure or the content structure is that the page kind naming follows its own logic. A page of kind I'm afraid we risk confusing the reader or worse having him question the naming logic or the veracity of the doc itself by using exemples. I therefore support simply stating what every kind is, using the best sentence we can. I'm sure those can still be improved. But then again, I might be wrong. Any third opinion would be welcome. |
fe001f9
to
4560631
Compare
@bep Do you want to move me as a PR to hugo core, or do we just wait for v0.38 release? |
Wait. Less work. |
Merge AFTER v0.38 release
Updates Output Formats documentation as per the change in v0.38.