Pass root in second argument to app().

The proposal is to remove the app’s root property and do the same passing it in the second argument to app().

Before

app({  init, view, state, actions, root }) 

After

app({ init, view, state, actions }, root) 

e.g.

app(
  { 
    init, 
    view, 
    state, 
    actions 
  }, 
  document.getElementById("app")
)

Pros

• Make app() props have to deal only with the state management side of things.
• Other libraries often do the same, e.g., P/React’s render(elm, root), Elm’s Elm.Main.embed(root), mithril’s m,mount(document.body, Counter) (the order is reversed here though), Vue’s $mount, etc.
• Cuts like 10 bytes. 😅

Cons

• There is nothing wrong with the current way of doing things.
• Adds like 10 bytes.
• Complicates the signature of HOAs. Thanks @okwolf.

[pspeter3]

I still want to make sure we have consistent reasoning for API changes. As we approach 1.0, the API will become locked.

[jorgebucaran]

The real (not really) question is, how do we feel about formatters like prettier making the code this:

app(
  { 
    init, 
    view, 
    state, 
    actions 
  }, 
  document.getElementById("app")
)