md-label with <md-input md-type="time"> should have the active class set up

[Ganbin]

I find out that if I use that kind of input : <md-input md-type="time" md-label="startTime" md-value.bind="Start Time"></md-input>

The Start Time label is on the same line as the --:-- empty value or even on a entered value like 12:15. I see that to put a active class on the label could be more appropriate with this kind of input because it always display something in the input.

I have take a look at the code of the input/input.js and I have added this 3 lines in the attached(), which make the trick :

if (this.mdType === 'time') {
    $(this.input.labels).addClass('active');
}

I don't know if you think it a good idea?

[Ganbin]

This is on chrome... but still have the md-input with md-label display issue on safari :(

[Thanood]

Thanks for the report, perfect timing. I will have to do some modifications anyway. 😄
As for #309, maybe I need to reactivate these lines: https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/blob/master/src/input/input.css#L1-L6

I'm not sure though.

[Thanood]

Reproduced it here:
https://gist.run/?id=7192c3cfc552a44f1ab07592d6a7f903

I wonder if Materialize itself has support for time inputs and if so, where we failed to transfer this. 😃

[Thanood]

The real problem with this issue is that Materialize doesn't update time inputs. 😕
https://github.com/Dogfalo/materialize/blob/master/js/forms.js#L6

[Thanood]

Added static "active" to label when it's a time input.