Classify for Dom.node

[Risto-Stevcev]

Something like Js.Json.classify and Js.Types.classify but for Dom.nodes

It would be based on similar tags as https://github.com/reasonml-community/bs-webapi-incubator/blob/master/src/Webapi/Webapi__Dom/Webapi__Dom__Types.re#L177-L190

But parameterized, ie:

type t =
  | Element of Dom.element
  | Text of Dom.text
  ...

let classify (node: Dom.node) : t = ...

[glennsl]

Not sure if you're asking for my approval of this, but if you are: I approve!

[Risto-Stevcev]

Yeah, but also just adding it here so I don't forget about it. I'm not sure when I'd be able to work on this

[yawaramin]

@Risto-Stevcev if you are planning to use instanceof, note that it's tricky because different windows have different prototype chains and instanceof doesn't work across windows. This will need to be accounted for. See e.g. https://github.com/reasonml-community/bs-webapi-incubator/blob/aad089aa2cd93e8884f53afb2507c2321265ff46/src/Webapi/Webapi__Dom/Webapi__Dom__Element.re#L8

[Risto-Stevcev]

Thanks for the heads up @yawaramin 🙂