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

Bindings near special characters fail in IE11 / IE10 #2077

Closed
kaestlyn opened this issue Jul 13, 2015 · 1 comment
Closed

Bindings near special characters fail in IE11 / IE10 #2077

kaestlyn opened this issue Jul 13, 2015 · 1 comment
Assignees
Labels

Comments

@kaestlyn
Copy link

I have a pretty simple component to validate this with a binding like so:

<link rel="import" href="../pathto/polymer.html"/>

<dom-module id="my-component">

    <template>
             <p>&copy;</p>
             <p>{{myText}}</p>
    </template>
</dom-module>
<script>
    Polymer({
        is: "my-component",
        properties: {
            myText: {
                type: String,
                value: 'Sample'
            }
        },
    });
</script>

Where myText is a property with any string value. In IE 11 this ends up getting rendered like:

image

Here's a plunker with the issue demonstration: http://plnkr.co/edit/5LDuhpPahqKgR25Z4CZG

I did work around it by making the © being a property as well and databinding it, but this seems a bit excessive for static text.

@sorvell
Copy link
Contributor

sorvell commented Jul 13, 2015

Thanks for finding this strange issue. It looks like in this case IE is splitting up the textNode that contains the binding into 2 nodes as can be seen here: http://jsbin.com/yizawodete/1/edit.

This happens as a result of loading the webcomponents-lite.js but it is unclear why. Since there are likely more cases that can cause text nodes to be split up like this Polymer's bind annotation parser should probably be robust to this scenario.

@sorvell sorvell self-assigned this Jul 13, 2015
@sorvell sorvell added the p1 label Jul 13, 2015
kevinpschaaf added a commit that referenced this issue Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants