Doctype option added - Fixes #19

[muratcorlu]

Caution: I set HTML5 as default doctype.

Usage:

{test: /\.jade$/, loader: 'jade?doctype=xhtml', exclude: /node_modules/}

Fixes #19

[colelawrence]

This is interesting, because I would argue that the doctype should not be
appended to the template unless explicitly specified. This is because in
every case where I use jade-loader, I'm using it for templates --- small
snippets of html for Angular templates or similar. It would be
inappropriate for these individual templates to each have a doctype.

On Thu, Sep 17, 2015, 3:19 AM Murat Çorlu notifications@github.com wrote:

Caution: I set HTML5 as default doctype.

Usage:

{test: /.jade$/, loader: 'jade?doctype=xhtml', exclude: /node_modules/}

---

You can view, comment on, or merge this pull request online at:

#24
Commit Summary

• Doctype option added

File Changes

• M index.js
  https://github.com/webpack/jade-loader/pull/24/files#diff-0 (1)

Patch Links:

• https://github.com/webpack/jade-loader/pull/24.patch
• https://github.com/webpack/jade-loader/pull/24.diff

—
Reply to this email directly or view it on GitHub
#24.

[MikaAK]

+1 for this. From the Jade Docs you can pass a doctype explicitly as well to be used for every file.

Not having a doctype causes some bugs in the non index file becuase things like div(ui-view) get parsed into <div ui-view='ui-view'></div> instead of <div ui-view></div>

[visnup]

+1. We have the same use case as @MikaAK where jade can't infer the doctype (html) from the snippets we send it and specifically <div ui-view></div> gets messed up. instead in our jade templates we end up having to write div( ui-view="" ) which isn't horrible, but noisy.

[MikaAK]

@sokra Any chance on reviewing this please? 😄

[sokra]

Thanks