Skip to content

Fix for issue #1, <meta> being before doctype#4

Open
elderfo wants to merge 1 commit intojsbin:masterfrom
elderfo:pr_meta_head
Open

Fix for issue #1, <meta> being before doctype#4
elderfo wants to merge 1 commit intojsbin:masterfrom
elderfo:pr_meta_head

Conversation

@elderfo
Copy link
Copy Markdown

@elderfo elderfo commented Dec 6, 2016 •
edited
Loading

Fix for issue #1, #2. Generates the following HTML:

With DOCTYPE & head:

<!DOCTYPE html>
<html>
<head>
<!--
Created using JS Bin
http://localhost:3000

Copyright (c) 2016 by anonymous (http://localhost:3000/duc/1/edit)

Released under the MIT license: http://jsbin.mit-license.org
-->
<meta name="robots" content="noindex">
  <meta charset="utf-8">
  <meta name="viewport" content="width=device-width">
  <title>JS Bin</title>
</head>
<body>
  <Div>stuff</div>
</body>
</html>

With DOCTYPE and no head (note: the meta tag is stripped):

<!DOCTYPE html>
<!--
Created using JS Bin
http://localhost:3000

Copyright (c) 2016 by anonymous (http://localhost:3000/duc/1/edit)

Released under the MIT license: http://jsbin.mit-license.org
-->
<html>
<body>
  <Div>stuff</div>
</body>
</html>

With no DOCTYPE and no head (note: the meta tag is also stripped):

<!--
Created using JS Bin
http://localhost:3000

Copyright (c) 2016 by anonymous (http://localhost:3000/duc/1/edit)

Released under the MIT license: http://jsbin.mit-license.org
-->
<html>
<body>
  <Div>stuff</div>
</body>
</html>

@remy, I intentionally did not worry about indenting here, I can add that if you would like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@elderfo