Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTML5: DOMQuery should use HTML5 parser #114

Open
technosophos opened this issue Jun 4, 2013 · 2 comments

Comments

Projects
None yet
2 participants
@technosophos
Copy link
Owner

commented Jun 4, 2013

The following functions need to be updated for HTML5 parsing and serializing.

  • parseXML
  • parseXMLFile
  • innerHTML
  • html()
  • writeHTML()
  • append()
  • prepend()
  • QueryPath::withHTML
  • htmlqp()

And need to add...

  • Documentation on use_parser=html5
  • html5qp() ?
  • html5()
  • writeHTML5()
@ptz-nerf

This comment has been minimized.

Copy link

commented Jan 23, 2016

Hello!
So, there is no way to insert a node with empty attribute?
For example, I got error trying to replace node with iframe with allowfullscreen attribute without value (embed code for yotube):

$node->replaceWith($iframeHtml);
DOMDocumentFragment::appendXML(): Entity: line 6: parser error : Specification mandate value for attribute allowfullscreen 

P.S. : Using html5qp().

@technosophos

This comment has been minimized.

Copy link
Owner Author

commented Jan 24, 2016

This looks like a big with HTML5-PHP. It is not allowing an empty attribute
value for that attribute.

On Sat, Jan 23, 2016, 11:32 AM Sergei notifications@github.com wrote:

Hello!
So, there is no way to insert a node with empty attribute?
For example, I got error trying to replace node with iframe with
allowfullscreen attribute without value (embed code for yotube):

$node->replaceWith($iframeHtml);

DOMDocumentFragment::appendXML(): Entity: line 6: parser error : Specification mandate value for attribute allowfullscreen

P.S. : Using html5qp().


Reply to this email directly or view it on GitHub
#114 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.