Martin Muñoz

It seems like `bits_read = "deku::rest.len()"` mostly works except that it cannot handle an empty vec.

``` Should not invoke element.focus() if the element already has focus ``` Calling element.focus() in this case should be fine. I think the issue is that we simulate focus events...

:tada:

Here's a list of events that should be instantiated with `{ composed: true }`: https://github.com/WICG/webcomponents/issues/513#issuecomment-224183937

This is a bug. We need to preserve both HTML and HBS comments.

Yes, they are 100% inert but for practical reasons it would be useful to have them in the AST, e.g. for future linting tools. We may opt instead to have...

The API of Shoelace's select doesn't use `selected` on the options so I wouldn't expect it to ever work with this helper. It makes sense to have a different set...

Looks like a bug. On the flip side, you can use Handlebars comments `{{!-- foo --}}` which should not have this problem. You shouldn't use unless you want to preserve...

Yes I'd love to ship an optimized bundled build, it's just not a priority right now. cc @eventualbuddha

@jkarsrud Yes that's how I'd do it. Keep in mind that AST transforms are not intended to be public yet (the AST is still in flux, awaiting for glimmer to...