platform.js Shadow DOM shim should work with attribute selectors

[jbdeboer]

e.g. a Component with the selector 'foo[bar]' will cause the Shadow DOM shim to fail.

This only effects browsers that need the shim.

[vsavkin]

It can be fixed only after https://github.com/Polymer/platform-dev/issues/44.

[vicb]

@vsavkin haven't you tried to fix the js and submit a PR ? What was the outcome ?
I don't think the linked issue is clear enough. It should also state what we need. I don't think we need any selector as we could easily add a class or an attribute. I thought that the only problem was that the selector is set to the tag name which is not unique to a (angular) component ?

[vsavkin]

@vicb I was going to, but it turned out to be a low-priority issue. So I did not do any work on it. I've updated the issue. Please check if it is clear now.

[vicb]

@vsavkin If think the description is much better now - especially the second part which propose a solution. What about going even further and propose an API. Then we are sure that what gets implemented fit our needs.

[vsavkin]

@vicb I've added an API proposal.

[vicb]

@vsavkin 👍 Have you tried to contact the polymer team about that ? It would great to get their opinion about this and an ETA if they agree

[vsavkin]

@vicb I haven't, but it is a good idea. I'll shoot them an email today.

[akserg]

Hi,

Any progress on that issue.

The monkeysecurity raised an issue on Angular Dart UI:

WARNING: 2015-01-02 18:11:00.002: WARNING: Failed to set up Shadow DOM shim for pagination[page][total-items].
InvalidCharacterError: The string contains invalid characters. 'pagination[page][total-items]' is not a valid attribute name.

It would be nice to have any feedback from you.

[vsavkin]

@akserg It looks like the issue has not been fixed in Platform.js. So it is not fixed in AngularDadrt either (cause we are emulating their approach). You can, however, define your own shim (as shown here https://github.com/angular/angular.dart/wiki/CSS-Shim), where you can, for instance, take the element part of the selector and use it for shimming.

[vicb]

https://github.com/Polymer/platform-dev/issues/44 has been moved to webcomponents/webcomponentsjs#28 (former repo is deprecated)