paper-textarea does not respect max-rows

[anthonytordillos]

Looks like the demo of this functionality in the elements catalog is broken: https://elements.polymer-project.org/elements/paper-input?view=demo:demo/index.html

[notwaldorf]

What browser are you seeing this on? When I use the textarea that has max-rows="4" set, then it starts scrolling once they're hit:

[anthonytordillos]

It looks like the behavior is correct when you have newlines in the text. The case I'm describing is when you have long text that wraps to multiple lines:

Based on the description of the maxRows property (The maximum number of rows this element can grow to until it scrolls.) I would expect it to scroll once the text has wrapped past maxRows lines.

[notwaldorf]

Ah, yes, I think that's related to PolymerElements/iron-autogrow-textarea#32

[anthonytordillos]

Yup. I agree with nloewen's conclusion that constrain should line wraps into account.

[theDataWhore]

so max-rows counts the new lines instead of the 'visible' row counts? i also noticed in the docs its maxRows not max-rows, but max-rows works.. i'm new to git so if i'm retreading something my apologies.

[notwaldorf]

You should use maxRows as a JavaScript property , and max-rows as an attribute in HTML: https://www.polymer-project.org/1.0/docs/devguide/properties.html#property-name-mapping

Closing this, since it seems to be an underlying iron-autogrow-textarea bug,

[faan11]

A solution would be to block the height of the iron-autogrow-textarea. It works but you have to compute the size in pixel.

:host{
      --paper-input-container-input:{
        max-height: 75px;
      }
 }