feat(input): #512 added autocomplete attribute

[zwalters]

No description provided.

[MaximBalaganskiy]

According to the definition the default value is "on" which is different to our default. Also, in most cases users won't use the attribute at all but we will add it to the DOM through the binding. I think it would be better to treat the attribute similar to placeholder. What do you recon?

[zwalters]

You're right. I'll set it similar to placeholder.

[zwalters]

How about we add it only if the value is off?

if(this.autocomplete == "off"){ this.input.setAttribute("autocomplete", this.autocomplete); }

[MaximBalaganskiy]

It actually can be any string, chrome for example understands a dozen or more

…

On Sat., 2 Jun. 2018, 1:02 am zwalters, ***@***.***> wrote: How about we add it only if the value is off? if(this.autocomplete == "off"){ this.input.setAttribute("autocomplete", this.autocomplete); } — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#516 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADimvBdeA4c8TuMH72RXGmX6X5JRwMmaks5t4VeOgaJpZM4UWtjQ> .

[zwalters]

Huh, that's neat. Good to know

[MaximBalaganskiy]

Fixes #512