Op maandag 5 december 2016 16:58:34 UTC+1 schreef Peter Damoc:
>
> I just pushed the latest changes and the functionality should be 
> equivalent. 
> https://github.com/pdamoc/polymer-exploration
>

Looks great! I would suggest a small change in the country update: to only 
reset the city selector if the country selected is different from the 
previous selection.

        CountryPicked country ->
            { model 
            | country = Just country
            , city = 
                if model.country /= Nothing && model.country /= Just 
country then
                    Nothing
                else
                    model.city
            }


Would have liked to get some sort of onSelectedChanged event working, but 
even though the event is in the elm-polymer library, I had no luck getting 
it to work.
Also, would have loved to make a pull request for you repository on github, 
but I worry that my pull request would include the gazillion bower and npm 
installs as well.
Sorry, I'm a noob at github too.

-- 
You received this message because you are subscribed to the Google Groups "Elm 
Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to