debois/elm-mdl

Textfield does not dispatch internal messages when user messages are installed

Closed this issue · 3 comments

I think this is the root cause of some Textfield issues, as mentioned in some of their comments. I've come up with a fix in 6feddc4. I have some second thoughts about the code and will leave this open for a while. Feedback appreciated.

@vipentti With regards to #164, is commit 6feddc4 the way to go? Does that mean we do not need to have the focusin/focusout polyfill anymore?

I think this is the way to go. For now, only Textfield uses this, see #338. I also removed the focusin/out polyfill as I think it is not needed anymore (27467d5).

dcbb commented

When will the fix to the labels not disappearing in Textfields shipped? I installed elm-mdl 8.1 with elm-package and I have the issue, but the here it's closed. Or am I missing something?