elm-community / typed-svg

Typed SVG library written for Elm

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Events.on documentation wrong and function seems wrong too

colwem opened this issue · comments

The documentation says both that the second arg is a Handler and that it's a Decoder a. The interface for all other packages like this is Decoder a It probably should be. But it isn't.|

Should maybe be

on : String -> Json.Decode.Decoder msg -> TypedSvg.Core.Attribute msg
on event decoder =
    VirtualDom.on event (VirtualDom.Normal decoder)

and not

on : String -> VirtualDom.Handler msg -> VirtualDom.Attribute msg
on =
    VirtualDom.on

Synced by #41 in favour of the code signature, not the documentation.