Adam Palaniuk
Adam Palaniuk
Hi @alexkrolick, it's a good question. JSDOM currently has shadow DOM support, with a few restrictions. The largest issue for testing right now is that this is almost entirely used...
@kentcdodds There's definitely a lot of complexity to work out, so I'm just being (overly) optimistic 🙂 we have done similar things at my company in the past (though only...
I haven't been able to revisit this at all, but I definitely appreciate the work others have done. I was thinking over the holidays that the relative complexity of the...
There should be no reason to walk the parents of the event target in that event handler, since what you really want is `this`. See #12
@cheonhyangzhang He probably bound a property with a default value of `null` to the attribute, which was never updated. This sounds like a bug in his consumer - why not...
Also, `if (typeof(this.tags) == 'undefined' || this.tags == null)` is redundant - `this.tags == null` will catch both `null` and `undefined`. But, shouldn't it be `this.tags instanceof Array`?
@barkerb1 It must have something to do with how you're modifying the `tags` property in your client code, but without seeing it, who knows why.
@cheonhyangzhang Exactly.
Using Storybook and its [viewport addon](https://github.com/storybookjs/storybook/tree/master/addons/viewport) makes this easy, and allows for the creation of explicitly-responsive demo pages (stories), which we've found useful.