emelie/zola
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Make live reload work with missing </body>

Browse source 
The HTML spec doesn’t require it, and I prefer to omit it. This has been
bothering me for ages, but I hadn’t gotten round to fixing it yet.

This can cause nominally invalid HTML to be emitted, if `</body>` was
omitted but `</html>` was present, but that’s unlikely to happen, and
this is for development purposes only, and the right thing will happen
anyway in all environments (per browser behaviour and spec).

I don’t think this warrants a changelog entry.
This commit is contained in:
Chris Morgan 2020-03-29 16:15:51 +05:30
parent cc64803553
commit cc01d3f82f
1 changed files with 9 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
components/site/src/lib.rs
View file

@ -626,15 +626,17 @@ impl Site {
} }
/// Inject live reload script tag if in live reload mode /// Inject live reload script tag if in live reload mode
fn inject_livereload(&self, html: String) -> String { fn inject_livereload(&self, mut html: String) -> String {
if let Some(port) = self.live_reload { if let Some(port) = self.live_reload {
return html.replace( let script = format!(
"</body>", r#"<script src="/livereload.js?port={}&amp;mindelay=10"></script>"#,
&format!( port,
r#"<script src="/livereload.js?port={}&amp;mindelay=10"></script></body>"#,
port
),
); );
if let Some(index) = html.rfind("</body>") {
html.insert_str(index, &script);
} else {
html.push_str(&script);
}
} }
html html