button.loading should be deprecated

[mattgallagher92]

It looks like the way daisyUI now represents buttons with loading spinners is by having a span with the loading class as a child of the button. The Feliz.DaisyUI equivalent is

Daisy.button.button [
    button.primary
    prop.children [
        Daisy.loading
    ]
]

which renders nicely:

However, the Feliz.DaisyUI API implies that something like this is sensible

Daisy.button.button [
    button.primary
    button.loading
]

which doesn't render well:

I recommend marking button.loading as deprecated, with a message advising use of Daisy.loading as a child instead.

[Dzoukr]

Good point, fixed now!