Allow Change of Menu Icon

[saucewaffle]

debois,

Needed to have multiple menus on a page so needed to be able to change what icon displays for them. Attached my version of Menu.elm with the changes, basically just imported Icon, setup an option and changed line 347 to set the Icon as whatever was sent in config.
Menu.elm.txt

Usage:
"
userMenu : Models.Model -> Html Messages.Msg
userMenu model =
div []
[
Menu.render MDL [0] model.mdl
[ Menu.bottomRight
, Menu.ripple
, Menu.icon "account_circle"
]
[ Menu.Item False True <| div [] [ text "My Profile" ]
, Menu.Item False True <| div [] [ text "Log In" ]
]

]
"

[debois]

Looks good! Could you submit a PR?

[saucewaffle]

"It seems you do not have permission to push your changes to this
repository" :(

[image: Inline image 1]

On Tue, Jun 21, 2016 at 11:39 AM, Søren Debois notifications@github.com
wrote:

Looks good! Could you submit a PR?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#61 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ARnHiVdpFnAGusfxpw4uTmgZsRCSbfvHks5qOAXCgaJpZM4I61y-
.

[debois]

GitHub has a very neat way to do this, but you have to do it in a particular way. See the GitHub docs here.

[debois]

Do you have time to upgrade the .txt file to an actual PR?

[saucewaffle]

Very sorry. I live in Cleveland and our team just won the basketball
championship. It's been crazy and I wasn't near a computer all day
yesterday. I'll have the PR to you today.

On Thu, Jun 23, 2016 at 6:31 AM, Søren Debois notifications@github.com
wrote:

Do you have time to upgrade the .txt file to an actual PR?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#61 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ARnHiRh2_sLqZMhWrR8LuGyDWdboGWvMks5qOmB7gaJpZM4I61y-
.

[debois]

Well, congratulations, then :)

I'll take a look at your PR.

[debois]

Closed by #65.