elm-community / typed-svg

Typed SVG library written for Elm

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

`TypedSvg.Attributes.version` takes a `Float`

zakvdm opened this issue · comments

Probably not a big deal, but seems like this should either be String or a new type.

May be best as a String.

Going with a String instead of introducing a new type. There are two official values this can take "1.0" and "1.1". However there are some variations that some browsers may accept and it seems nicer to allow the user to put anything in here to allow for experimentation.

@rupertlssmith @zakvdm
I suppose this typed version would be better:

type Version 
    = Version_1_0
    | Version_1_1
    | VersionCustom String