feat(input): allow skip hidden inputs

[devversion]

inputs with type hidden will be skipped by the input-container

breaking

fixes #2153

[topherfangio]

I think this warrants a note in the docs about how we skip inputs. Otherwise, LGTM.

[devversion]

@topherfangio - I added a note to the <md-input-container> directive. Is that note enough?

PS: CI is still failing due Karma (will be fixed in [email protected])

[topherfangio]

Let's change the wording to this:

A single <md-input-container> should contain only one <input> element, otherwise it will throw an error.

Exception: Hidden inputs (<input type="hidden" />) are ignored and will not throw an error, so
you may combine these with other inputs.

[devversion]

If I got your new wording right (with the divider and so on), it should look like this? (bold exception seems to be smaller as the actual text is).

Still needs work? @topherfangio

[devversion]

@topherfangio - Can you review the documentation message?

[topherfangio]

@ThomasBurleson LGTM; not sure which milestone you want this in. Can you please take a look?

[topherfangio]

I'm going to go ahead and delay this until 1.1.0 as it could affect some of our existing users.

@devversion Can you please amend the commit message with a BREAKING CHANGE note about how we now skip inputs. We do it in the commit message because our Changelog process automatically pulls from there when noting breaking changes.

[devversion]

@topherfangio Done - Just updated the PR description too, so you can see it there.