Adding the AMP boolean attributes to the attributes module

I am trying to render an AMP HTML page using snabbdom. AMP has some custom boolean attributes: amp, ⚡, amp-boilerplate (might be more, I don’t know yet).

Would it be fine to add those attributes to the list of boolean attributes here

Note: currently snabbdom renders those as amp="true" and that does not pass the AMP validator.

---

I can work on a fork and try to make all the necessary changes that need to be done for snabbdom to be able to render AMP-compliant pages

[paldepind]

Thank you for reporting this issue @zkochan. The latest release now handles custom boolean attributes seamlessly.

h("div", { attrs: { amp: true} } results in <div amp>.

[fix-fix]

Yes, but I don’t think that module should replace attributes module, but instead there should be a special way for providing AMP (and maybe some other non-standard) attributes to VNodes (not in the data property), that can be used alongside default attributes module.