Components with with names of native elements throw compilation error

[Exelord]

Creating components with names of some native elements like Script, Title etc results with an error :
Unclosed element `Title` - glimmer-engine
when using them as a block:

<Title>
  It is closed
</Title>

Is it expected behavior?

Tested in Ember 3.13

[rwjblue]

Creating components with names of self-closing native elements like Script, Title

Not sure what you mean here. Script and title both require closing tags (e.g. are not “self closing”).

Is it expected behavior?

Hmm, no I don’t think this is expected. Is it limited to title/script/noscript/style though? Does (for example) <Span>Foo</Span> have an issue?

[Exelord]

Not sure what you mean here. Script and title both require closing tags (e.g. are not “self closing”).
My mistake sorry

Hmm, no I don’t think this is expected. Is it limited to title/script/noscript/style though? Does (for example) Foo have an issue?

Not all of them. Im not sure about Span but Div works fine.

also you can use title as <Title />. It just doesn't work as block.

[rwjblue]

I suspect the issue is with the changes from tildeio/simple-html-tokenizer#69. Since (in the spec) element tag names are case insensitive, this patch does tagName.toLowerCase() to normalize. Unfortunately, this means that we treat <Title> and <Style> the exact same way that we treat <title> and <style>.

In addition, that PR lowercases the tag name and looks for a closing tag that is also lowercase (IOW it will match <Title> but not match </Title>). I think this is the reason for the specific error you see here.

[rwjblue]

The fix is most likely to fix that PR over in simple-html-tokenizer to specifically only look for the lowercase variants (removing that .toLowerCase() call).

[rwjblue]

@CvX - Do you think you might have any time to poke at this? To confirm my suspicion and/or send in a patch?

[dcyriller]

@rwjblue I gave it at try here: tildeio/simple-html-tokenizer#81