HTML head customisation with head.hbs#1206
Conversation
ehuss
left a comment
There was a problem hiding this comment.
Could you also update the documentation in the theme docs for this and header.hbs?
There was a problem hiding this comment.
I'm not sure why this is added.
There was a problem hiding this comment.
Ah, I see. I just pushed a change to rewrite it: #1214. I usually prefer unrelated changes like this to be separate.
There was a problem hiding this comment.
I'd prefer to avoid unrelated formatting changes. Can you remove all the whitespace changes and formatting changes?
There was a problem hiding this comment.
Code auto-formatting has been reverted.
|
@Dylan-DPC does this seem like a reasonable idea to you? I suspect there might be some longer term issues with making it easier to extend the templates (#353 ?), but this seems like a simple, incremental improvement. |
|
i don't see any issue with this |
|
Documentation added. Comments answered. Thanks for your reviews. |
Create a `index.hbs` template file to render additional HTML <head> attributes
|
The documentation might be confusing. It says:
I think it should say "under" or "inside" the src folder because that's what worked for me. Not having |
HTML head customisation with `head.hbs`

Allow to define custom attributes within the HTML tag thanks to a
head.hbstemplate file.A typical use case is the addition of mobile capabilities for PWA deployment for example to render the doc in fullscreen once added to the home screen (Android or iOS):