Several error in the documentation

[dgabrielli76]

Description

There are several small errors in the documentation of this element.

Just one example:

Expected outcome

</script>

Actual outcome

</script>

I will fix this errors and I will propose a PR soon to fix this issue.

[notwaldorf]

That </script> is needed so that the docs show up correctly in the catalog, but I guess that makes them look silly in the readme.

@garlicnation @arthurevans any ideas?

[arthurevans]

I don't think that you should need to escape that one bracket (weird that it's the second-to-last-one, too). But um ah I'm probably wrong?

[notwaldorf]

That one bracket makes </script> and hydrolysis stops parsing anything after 😎

[arthurevans]

On a scale of 1 to bad that seems really bad. @garlicnation is this a known issue with hydrolysis?

[felixzapata]

Hi @notwaldorf @arthurevans @garlicnation , in order to help, I think that the tag should be written like this '</\script>'. This change does not fix the issue but, I 've made a test using the demo of the `iron-doc-viewer' component and, if you use this syntax, the tag is parsed ok.

I think that hydrolysis should take care of this syntax, escaping the slash.

[felixzapata]

Hi again @notwaldorf @arthurevans @garlicnation, another idea.. after some tests over the iron-doc-viewer component, if you replace:

this.toggleAttribute('abstract', this.descriptor.abstract);

with this

this.set('descriptor.desc', this.descriptor.desc.replace(/\/\\script/gi, '/script'));
this.toggleAttribute('abstract', this.descriptor.abstract);

That is to say, an small fix to find and replace a pattern for the script and replace with the correct value.

After that, you can view the documentation with the script tag fixed:

But, really I am not sure if this is the best place to make the fix. I mean, maybe this small parse should be inside hydrolysis.