Skip to content

Commit dcaf1c7

Browse files
committed
Change namespaceHTMLElements to default to True, so we follow the spec by default. Add a warning about us being broken when set to False.
1 parent 4699f40 commit dcaf1c7

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/html5lib/html5parser.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
from constants import tokenTypes, ReparseException, namespaces
2222

2323
def parse(doc, treebuilder="simpletree", encoding=None,
24-
namespaceHTMLElements=False):
24+
namespaceHTMLElements=True):
2525
tb = treebuilders.getTreeBuilder(treebuilder)
2626
p = HTMLParser(tb, namespaceHTMLElements=namespaceHTMLElements)
2727
return p.parse(doc, encoding=encoding)
@@ -32,7 +32,7 @@ class HTMLParser(object):
3232

3333
def __init__(self, tree = simpletree.TreeBuilder,
3434
tokenizer = tokenizer.HTMLTokenizer, strict = False,
35-
namespaceHTMLElements = False):
35+
namespaceHTMLElements = True):
3636
"""
3737
strict - raise an exception when a parse error is encountered
3838

src/html5lib/treebuilders/_base.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import warnings
12
from html5lib.constants import scopingElements, tableInsertModeElements
23
try:
34
frozenset
@@ -114,6 +115,7 @@ def __init__(self, namespaceHTMLElements):
114115
self.defaultNamespace = "http://www.w3.org/1999/xhtml"
115116
else:
116117
self.defaultNamespace = None
118+
warnings.warn(u"namespaceHTMLElements=False is currently rather broken, you probably don't want to use it")
117119
self.reset()
118120

119121
def reset(self):

0 commit comments

Comments
 (0)