circuithub/elm-html-shorthand

Html shorthand for Elm

elm-html-shorthand's Issues

ImgParam

I guess ImgParam have to be as follows

type alias ImgParam = 
    { class : ClassString
    , src : UrlString
    , width : Maybe Int
    , height : Maybe Int
    , alt : Maybe String
    }

Version 10.0.0 publishing

Hello, @rehno-lindeque.

Looks like version 10.0.0 has not been published yet, but it appears in elm-bootstrap-html dependencies.

BTW, I think it would be better if new version should be lower than 10.0.0, because there's no major API changes, as I see.

Should we use short or long names for derived elements?

For example, <a> and <button> has the idiomatic forms:

a' : UrlString -> TextString -> Html
button' : TextString -> Signal.Message -> Html

should a "button" link (an anchor turned into a button via the onClick event) look like:

aButton' : TextString -> Signal.Message -> Html

or

buttonLink' : TextString -> Signal.Message -> Html

or even one of these?

a'button : TextString -> Signal.Message -> Html
button'link : TextString -> Signal.Message -> Html

---

Similarly <input> has several possible idiomatic forms:

inputField' : FieldUpdate -> Html
inputRadio' : FieldUpdate -> Html
inputCheckbox' : FieldUpdate -> Html

or

field' : FieldUpdate -> Html
radio' : FieldUpdate -> Html
checkbox' : FieldUpdate -> Html

or

input'field : FieldUpdate -> Html
input'radio : FieldUpdate -> Html
input'checkbox : FieldUpdate -> Html

---

Note that all of these would all have classy forms also:

aButtonc : TextString -> Signal.Message -> Html -- possibly buttonLinkc
inputFieldc : FieldUpdate -> Html
inputRadioc : FieldUpdate -> Html
inputCheckboxc : FieldUpdate -> Html

versus

buttonlinkc : TextString -> Signal.Message -> Html
fieldc : FieldUpdate -> Html
radioc : FieldUpdate -> Html
checkboxc : FieldUpdate -> Html

versus

acbutton : TextString -> Signal.Message -> Html
inputcfield : FieldUpdate -> Html
inputcradio : FieldUpdate -> Html
inputccheckbox : FieldUpdate -> Html

Seems that v 7.0.0 won't build with Elm 0.15.1

I got this message when attempting to install my package which uses elm-html-shorthand:

Downloading circuithub/elm-bootstrap-html
Downloading circuithub/elm-html-extra
Downloading circuithub/elm-html-shorthand
Downloading elm-lang/core
Downloading evancz/elm-html
Downloading evancz/elm-http
Downloading evancz/virtual-dom
Packages configured successfully!
Detected errors in 1 module.
## ERRORS in dependency circuithub/elm-html-shorthand 7.0.0 ####################

This error probably means that the 'circuithub/elm-html-shorthand' has some
package constraint that is too permissive. You should definitely inform the
maintainer to get this fixed and save other people from this pain.

In the meantime, you can attempt to artificially constrain things by adding
some extra constraints to your elm-package.json though that is not
the long term solution.

My own elm-package.json looks like this:

{
    "version": "1.0.0",
    "summary": "helpful summary of your project, less than 80 characters",
    "repository": "https://github.com/USER/PROJECT.git",
    "license": "BSD3",
    "source-directories": [
        ".",
        "src"
    ],
    "exposed-modules": [],
    "dependencies": {
        "circuithub/elm-bootstrap-html": "6.3.0 <= v < 7.0.0",
        "circuithub/elm-html-shorthand": "8.0.0 <= v < 9.0.0",
        "elm-lang/core": "2.1.0 <= v < 3.0.0",
        "evancz/elm-html": "3.0.0 <= v < 4.0.0",
        "evancz/elm-http": "1.0.0 <= v < 2.0.0"
    },
    "elm-version": "0.15.0 <= v < 0.16.0"
}

I'm not sure what the issue is, a first look at the deps for 7.0.0 seem okay. I did notice one bit of strangeness: edad2a8 in this commit it looks like the version goes from 8.0.1 to 7.0.0. Am I reading that correctly? Would that cause problems?

Thanks!

encodeClass issues

All the classes are converted to lower case with your package which seems my styles not to work correctly.

I use upper-case letters in my classes because I use BEM methodology.

Elm 0.17 upgrade?

Just wondered if a 0.17 update was being worked on