Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

text-expand component: maxLength not working #617

Closed
blackbaud-joshgerdes opened this issue Apr 21, 2017 · 2 comments
Closed

text-expand component: maxLength not working #617

blackbaud-joshgerdes opened this issue Apr 21, 2017 · 2 comments

Comments

@blackbaud-joshgerdes
Copy link
Contributor

Expected behavior

To display more text than the default (200 characters) if maxLength property set.

Actual behavior

Just truncates text to the default (200 characters).

Steps to reproduce

Setup component with maxLength property.

Plunker (see example SKY UX 2 plunker template at: https://embed.plnkr.co/cPPULcTfyzUm2rouCgk2/

@Blackbaud-TrevorBurch
Copy link
Member

https://plnkr.co/edit/a2dG9kOTqvIeRwhJLAID?p=preview

I'll look at this. It appears to be a timing issue. When the text input is set and thus calls the setup function it does not yet have the value of the maxLength input and thus it doesn't setup correctly. The plunker above shows that if I change the text after initial load it will pick up the correct maxLength.

@Blackbaud-PatrickOFriel
Copy link
Contributor

In 2.0.0-beta.22. Thanks @Blackbaud-TrevorBurch !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants