Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

fix(node-document): create conditional for non html component #614

Closed
wants to merge 5 commits into from
Closed

fix(node-document): create conditional for non html component #614

wants to merge 5 commits into from

Conversation

MarkPieszak
Copy link
Contributor

Fixes #347

@MarkPieszak
Copy link
Contributor Author

So is this one not helpful to fix the bigger issue (with Core handling the head title meta properties itself?)
@gdi2290



/*
// Build entire doc <!doctype><html> etc
if (documentHtml.indexOf('<html>') > -1 && documentHtml.indexOf('</html>') > -1) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is still a problem with html tags that have attributes like <html ⚡> or <html lang="en">

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Totally right. Should I clean it up to test for just <html, maybe even look for doctype as well? @gdi2290

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use a regexp looking for any case of doctype

@PatrickJS
Copy link
Contributor

might be supported in ng4
angular/angular#13822

@PatrickJS PatrickJS closed this Jan 8, 2017
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support rendering just a single component rather than a complete HTML document
2 participants